Add support for Audio and Video summarization to Docsum#865
Merged
ashahba merged 46 commits intoopea-project:mainfrom Nov 13, 2024
Merged
Add support for Audio and Video summarization to Docsum#865ashahba merged 46 commits intoopea-project:mainfrom
ashahba merged 46 commits intoopea-project:mainfrom
Conversation
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Codecov ReportAttention: Patch coverage is
|
ashahba
requested changes
Nov 8, 2024
comps/dataprep/multimedia2text/video2audio/video2audio_microservice.py
Outdated
Show resolved
Hide resolved
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
for more information, see https://pre-commit.ci
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
for more information, see https://pre-commit.ci
4 tasks
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
for more information, see https://pre-commit.ci
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Collaborator
|
@MSCetin37, |
Signed-off-by: Mustafa <mustafa.cetin@intel.com>
Collaborator
@lvliang-intel the 3 Dockerfiles in the PR are already part of the build in that file so we are good. |
6 tasks
Collaborator
|
The audio2text service is almost the same as the asr service except the returned data format. Why can't we combine these 2 services into one? |
madison-evans
pushed a commit
to SAPD-Intel/GenAIComps
that referenced
this pull request
May 12, 2025
…#865) * v2a services Signed-off-by: Mustafa <mustafa.cetin@intel.com> * add a2t - llm Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update whisper serve Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates Signed-off-by: Mustafa <mustafa.cetin@intel.com> * add data service Signed-off-by: Mustafa <mustafa.cetin@intel.com> * gateway Signed-off-by: Mustafa <mustafa.cetin@intel.com> * clean gateway & orchestrator Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates Signed-off-by: Mustafa <mustafa.cetin@intel.com> * adding functional tests Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates Signed-off-by: Mustafa <mustafa.cetin@intel.com> * updates read me file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * name changes Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update readme file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update readme file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update readme file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update readme file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update readme file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update max token option Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update the test files Signed-off-by: Mustafa <mustafa.cetin@intel.com> * readme updtes Signed-off-by: Mustafa <mustafa.cetin@intel.com> * readme updtes Signed-off-by: Mustafa <mustafa.cetin@intel.com> * clean code Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update dataprep-compose-cd.yaml file Signed-off-by: Mustafa <mustafa.cetin@intel.com> * merge and sync Signed-off-by: Mustafa <mustafa.cetin@intel.com> * merge and sync gateway Signed-off-by: Mustafa <mustafa.cetin@intel.com> * adding the copyright header Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update the end of file char Signed-off-by: Mustafa <mustafa.cetin@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update gateway Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update gateway-docsum Signed-off-by: Mustafa <mustafa.cetin@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix test files Signed-off-by: Mustafa <mustafa.cetin@intel.com> * fix test files Signed-off-by: Mustafa <mustafa.cetin@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * merge & sync Signed-off-by: Mustafa <mustafa.cetin@intel.com> * Update dataprep-compose.yaml * update test Signed-off-by: Mustafa <mustafa.cetin@intel.com> * update dataprep-compose Signed-off-by: Mustafa <mustafa.cetin@intel.com> --------- Signed-off-by: Mustafa <mustafa.cetin@intel.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com> Co-authored-by: ZePan110 <ze.pan@intel.com>
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
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
Extend the current Document Summarization Application by incorporating video and audio summary features. This enhancement will enable the application to summarize video and audio content in addition to text documents, thereby broadening its utility and applicability.
Issues
https://github.com/opea-project/docs/blob/main/community/rfcs/24-06-21-OPEA-001-DocSum_Video_Audio.md
Type of change
List the type of change like below. Please delete options that are not relevant.
Tests
Whisper Service
Run the following command to validate the Whisper Service:
Expected output:
Audio2Text Service
Run the following command to validate the Audio2Text Service:
Expected output:
Note: The
idvalue will be different.Video2Audio Service
Run the following command to validate the Video2Audio Service:
Expected output:
Multimedia2Text Service
Run the following command to validate the Multimedia2Text Service:
Expected output:
cd GenAIComps/tests bash tests/dataprep/test_dataprep_multimedia.sh