Skip to content

Add short descriptions to the images OPEA publishes on Docker Hub #1637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Apr 1, 2025

Conversation

zhanmyz
Copy link
Contributor

@zhanmyz zhanmyz commented Mar 7, 2025

Description

Add the description of the image in the repository GenAIComps and GenAIExamples to the corresponding Docker Hub with the recommended dockerhub-description tool

Issues

Fixes opea-project/docs#316

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

N/A

Tests

Describe the tests that you ran to verify your changes.

Copy link

github-actions bot commented Mar 7, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

  • .github/workflows/dockerhub-description.yml

@letonghan letonghan added the OPEAHack Issue created for OPEA Hackathon label Mar 17, 2025
@ftian1
Copy link
Collaborator

ftian1 commented Mar 17, 2025

pls fix preci failure @zhanmyz

@chensuyue
Copy link
Collaborator

Let me know if you need more discussion or inputs from my side.

@chensuyue chensuyue requested a review from ashahba March 24, 2025 06:41
@chensuyue
Copy link
Collaborator

Hope we can finish this PR by v1.3 release.

@yinghu5
Copy link
Collaborator

yinghu5 commented Mar 25, 2025

Hi @zhanmyz , thank you a lot for the contribution. Just remind please help check the comments

2. Update the trigger strategy of dockerhub description from on push to schedule weekly
@yinghu5 yinghu5 added this to the v1.3 milestone Apr 1, 2025
@yinghu5 yinghu5 added this to OPEA Apr 1, 2025
Copy link
Collaborator

@ashahba ashahba left a comment

Choose a reason for hiding this comment

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

In general the README.md files may contain relative hyperlinks that will not resolve properly when clicked from Dockerhub since the relative path is on GitHub, but I think this is already a great start and we can fix those in a subsequent PR.

Once the tests pass, we should get this merged and then focus on improvements.

LGTM.

@chensuyue
Copy link
Collaborator

hyperlink issue due to the code change of https://github.com/vllm-project/vllm main branch, let's freeze the link as https://github.com/vllm-project/vllm/blob/v0.6.1/Dockerfile.openvino .

@chensuyue
Copy link
Collaborator

pre-commit.ci issue means your new workflow miss the license header.

@chensuyue
Copy link
Collaborator

This should be the first PR for test, after this PR merged. We should optimize the PR with strategy for loop to reduce the maintain effort.

@yinghu5 yinghu5 self-requested a review April 1, 2025 07:33
@chensuyue chensuyue merged commit 68747a9 into opea-project:main Apr 1, 2025
11 checks passed
@github-project-automation github-project-automation bot moved this to Done in OPEA Apr 1, 2025
@chensuyue
Copy link
Collaborator

username: ${{ secrets.DOCKERHUB_USER }}
password: ${{ secrets.DOCKERHUB_TOKEN }}
repository: opea/audioqna
short-description: "The docker image served as an audioqna gateway and using language modeling to generate answers to user queries by converting audio input to text, and then using text-to-speech (TTS) to convert those answers back to speech for interaction."
Copy link
Contributor

Choose a reason for hiding this comment

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

These descriptions are for DockerHub, they do not need to mention they are docker images, that's already known.

And past tense (served, acted, powered, exposed) sounds really odd in these description. Don't those images work anymore?

Something like this would be more to the point for all of these descriptions:

AudioQnA gateway. Uses language modeling to interact with the user, first by converting audio input queries to text, and then using text-to-speech (TTS) to convert those answers back to speech.

chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request May 16, 2025
…ea-project#1637)

Signed-off-by: zhanmyz <yazhan.ma@intel.com>
Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OPEAHack Issue created for OPEA Hackathon
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add short descriptions to the images OPEA publishes on Docker Hub
7 participants