Update AgentQnA and DocSum for Gaudi Compatibility#1777
Merged
ashahba merged 1 commit intoopea-project:mainfrom Apr 16, 2025
Merged
Update AgentQnA and DocSum for Gaudi Compatibility#1777ashahba merged 1 commit intoopea-project:mainfrom
ashahba merged 1 commit intoopea-project:mainfrom
Conversation
Mahathi-Vatsal
requested review from
XinyaoWa,
letonghan,
lkk12014402 and
minmin-intel
as code owners
Dependency Review✅ No vulnerabilities or license issues found.Scanned FilesNone |
Collaborator
|
Hi @Mahathi-Vatsal could you please fix the DCO issue? Thanks. |
ashahba
approved these changes
Apr 9, 2025
Collaborator
ashahba
left a comment
ashahba
left a comment
There was a problem hiding this comment.
LGTM!
Thanks @Mahathi-Vatsal
minmin-intel
reviewed
Apr 9, 2025
Mahathi-Vatsal
force-pushed
the
fixes_for_gaudi
branch
2 times, most recently
from
ca40941 to
b3cd090
Compare
Mahathi-Vatsal
requested review from
Spycsh,
WenjiaoYue,
ZePan110,
chensuyue,
ftian1,
lvliang-intel and
rbrugaro
as code owners
ashahba
requested changes
Apr 10, 2025
Collaborator
ashahba
left a comment
ashahba
left a comment
There was a problem hiding this comment.
I didn't pay close attention this afternoon to commit list, but now that I look closer I see many commits added by this PR.
@Mahathi-Vatsal do you want to update your fork and reset your branch so that only a few commits (ideally one only since this is a very small PR) that truly belong to you show in the commit list?
Thanks.
Mahathi-Vatsal
force-pushed
the
fixes_for_gaudi
branch
from
c7cd5b4 to
a560c27
Compare
Contributor
Author
|
@ashahba : I cleaned up the commits and also included the suggested steps in run_ingest_data.sh. I will create a PR to github.io documentation to include note in the README to the two required packages for AgentQnA. |
Collaborator
|
@Mahathi-Vatsal, |
Signed-off-by: Mahathi Vatsal <[email protected]>
Mahathi-Vatsal
force-pushed
the
fixes_for_gaudi
branch
from
42431c2 to
22c5ec0
Compare
lvliang-intel
approved these changes
Apr 16, 2025
4 tasks
chyundunovDatamonsters
pushed a commit
to chyundunovDatamonsters/OPEA-GenAIExamples
that referenced
this pull request
May 16, 2025
Signed-off-by: Mahathi Vatsal <[email protected]> Signed-off-by: Chingis Yundunov <[email protected]>
letonghan
pushed a commit
that referenced
this pull request
Sep 17, 2025
* Add postgres health check for text2sql API "/v1/postgres/health" is missing in text2sql microservice, which is required by UI in GenAIExample. This commit adds the api and fix several typos in README. This PR should fix #1776. Signed-off-by: PeterYang12 <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Signed-off-by: PeterYang12 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Ying Hu <[email protected]>
cogniware-devops
pushed a commit
to Cogniware-Inc/GenAIExamples
that referenced
this pull request
Dec 19, 2025
Signed-off-by: Mahathi Vatsal <[email protected]> Signed-off-by: cogniware-devops <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Updated AgentQnA and DocSum components to enable smooth execution on Gaudi.
Issues
n/a — no specific issue or RFC linked to this PR.
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
The following dependencies were explicitly installed via pip to ensure proper runtime:
Both are commonly used and lightweight Python libraries.
Tests