Skip to content

Adding the two missing packages for ingest script#1822

Merged
louie-tsai merged 2 commits intoopea-project:mainfrom
ashahba:ashahba/AgentQnA-ingest
Apr 16, 2025
Merged

Adding the two missing packages for ingest script#1822
louie-tsai merged 2 commits intoopea-project:mainfrom
ashahba:ashahba/AgentQnA-ingest

Conversation

@ashahba
Copy link
Copy Markdown
Collaborator

@ashahba ashahba commented Apr 16, 2025

Description

I just merged #1777 but the actual required packages did not make it through!

Issues

Please see #1777

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

Neither requests nor tqdm are new.

Tests

AgentQnA tests

@ashahba
Adding the two missing packages ingest script
ef84f93 
Signed-off-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 16, 2025 05:13
Copilot AI reviewed Apr 16, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • AgentQnA/retrieval_tool/run_ingest_data.sh: Language not supported

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 16, 2025 •
edited
Loading

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

@ashahba ashahba added this to the v1.3 milestone Apr 16, 2025
@ashahba ashahba added the v1.3 label Apr 16, 2025
@joshuayao joshuayao added this to OPEA Apr 16, 2025
@joshuayao joshuayao moved this to In review in OPEA Apr 16, 2025
@ashahba ashahba changed the title Adding the two missing packages ingest script Adding the two missing packages for ingest script Apr 16, 2025
mkbhanda
mkbhanda approved these changes Apr 16, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@mkbhanda mkbhanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@louie-tsai louie-tsai merged commit a7ef833 into opea-project:main Apr 16, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in OPEA Apr 16, 2025
@ashahba ashahba deleted the ashahba/AgentQnA-ingest branch April 16, 2025 19:03
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request May 16, 2025
@ashahba @chyundunovDatamonsters
Adding the two missing packages for ingest script (opea-project#1822)
4a6a675 
Signed-off-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com>
Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
letonghan pushed a commit that referenced this pull request Sep 17, 2025
@ZePan110
Reduce chunk_size and add timeout to fix issue (#1822)
27e2a8a 
* Fix issue
* Add timeout for UT script.

Signed-off-by: ZePan110 <ze.pan@intel.com>

---------

Signed-off-by: ZePan110 <ze.pan@intel.com>
cogniware-devops pushed a commit to Cogniware-Inc/GenAIExamples that referenced this pull request Dec 19, 2025
@ashahba @cogniware-devops
Adding the two missing packages for ingest script (opea-project#1822)
05c78d4 
Signed-off-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com>
Signed-off-by: cogniware-devops <ambarish.desai@cogniware.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@mkbhanda mkbhanda mkbhanda approved these changes

@louie-tsai louie-tsai louie-tsai approved these changes

@lkk12014402 lkk12014402 Awaiting requested review from lkk12014402 lkk12014402 is a code owner

@minmin-intel minmin-intel Awaiting requested review from minmin-intel minmin-intel is a code owner

Assignees

No one assigned

Labels

v1.3

Projects

OPEA
Status: Done

Milestone

v1.3

Development

Successfully merging this pull request may close these issues.

5 participants

@ashahba @mkbhanda @louie-tsai @joshuayao