Skip to content

Conversation

@artem-astafev
Copy link
Contributor

Description

Changes for MultimodalQnA example deployment to AMD ROCm systems.

Type of change

  • Added compose example for MultimodalQnA deployment on AMD ROCm systems.
  • Added Readme file for MultimodalQnA compose deployment example on AMD ROCm systems.
  • Added CI tests for MultimodalQnA compose deployment example on AMD ROCm systems.

Tests

For manual test s go to GenAIExamples/MultimodalQnA/tests dir and run test_compose_on_rocm.sh script (./test_compose_on_rocm.sh)

@artem-astafev artem-astafev force-pushed the feature/MultimodalQnA-add-compos-example-for-amd-rocm branch from 15ea852 to 95f9b07 Compare December 6, 2024 08:40
Copy link
Collaborator

@letonghan letonghan left a comment

Choose a reason for hiding this comment

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

Hi @artem-astafev , thanks for your contribution.
Please check the comment here and refine the set_env part in your readme.

@artem-astafev
Copy link
Contributor Author

artem-astafev commented Dec 13, 2024

Hi @artem-astafev , thanks for your contribution. Please check the comment here and refine the set_env part in your readme.

Hi @letonghan , I've removed manual environment set part and add set_env.sh description in Readme.md in last 2 commits(c7291ce and 0ecb8b5).

I can't get what else can be change in Readme.md for better description. Is these enough or something else need to be changed ?

@artem-astafev
Copy link
Contributor Author

CI Tests starts to fail because of backend microservice could start with error:

File "/home/user/multimodalqna.py", line 10, in <module> from comps import Gateway, MegaServiceEndpoint, MicroService, ServiceOrchestrator, ServiceType ImportError: cannot import name 'Gateway' from 'comps' (/home/user/GenAIComps/comps/__init__.py)

Probably because of opea-project/GenAIComps#979 merge.

@github-actions
Copy link

github-actions bot commented Dec 17, 2024

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

…tems

Signed-off-by: artem-astafev <a.astafev@datamonsters.com>
@artem-astafev artem-astafev force-pushed the feature/MultimodalQnA-add-compos-example-for-amd-rocm branch from 7573f4a to a4ef17c Compare December 18, 2024 09:16
Copy link
Collaborator

@letonghan letonghan left a comment

Choose a reason for hiding this comment

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

lgtm

@letonghan letonghan merged commit 236ea6b into opea-project:main Dec 18, 2024
14 checks passed
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request Dec 20, 2024
opea-project#1233)

Signed-off-by: artem-astafev <a.astafev@datamonsters.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
letonghan added a commit that referenced this pull request Sep 17, 2025
* Update Retrievers README

Add a main README in comps/retrievers/README.md, and contains 9 READMEs
for each vectordb.

Signed-off-by: letonghan <letong.han@intel.com>
cogniware-devops pushed a commit to Cogniware-Inc/GenAIExamples that referenced this pull request Dec 19, 2025
opea-project#1233)

Signed-off-by: artem-astafev <a.astafev@datamonsters.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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants