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.

Sign up for GitHub

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

Jump to bottom

Docsum #1095

Open

MSCetin37 wants to merge 162 commits into opea-project:main from MSCetin37:docsum

+1,385 −270

MSCetin37 commented Nov 8, 2024 •

edited

Loading

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.

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

GenAIExamples/DocSum/tests/test_compose_on_gaudi.sh
GenAIExamples/DocSum/tests/test_compose_on_xeon.sh
GenAIExamples/DocSum/tests/test_gmc_on_gaudi.sh
GenAIExamples/DocSum/tests/test_gmc_on_xeon.sh
GenAIExamples/DocSum/tests/test_manifest_on_gaudi.sh
GenAIExamples/DocSum/tests/test_manifest_on_xeon.sh

MSCetin37 added 30 commits

October 14, 2024 11:56


          initial

40246c1

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update the compose file

10912e4

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

fab8069

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          add docsum data

c9c1a4c

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

8dee06a

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

b49f2c4

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

53abcc2

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

06d1ead

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          add functional test and data

1a2eb00

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update the bash test functions

3faf2f9

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          test_compose

58617f5

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update the test function

381d782

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          add ui

0ee2c4c

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update the py test file

d6b73b2

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update docsum name to multimedia

a9959b4

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update hpu

65ee6ca

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          gaudi updates

64835a0

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update gaudi test

d17eb80

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          updates

6ff021c

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          add test files

a39a273

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          gaudi data updates

70b2446

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

6f46d78

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

2d8621a

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

2a50c1a

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

4418a5f

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

eebff1f

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

972ca0f

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

7e83d71

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

70d145b

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          update read me file

e7d6ca0

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

Collaborator

lianhao commented Nov 15, 2024 •

edited

Loading

My previous comments somehow got hided, so let me put here:

The k8s manifest test is actually failed, but I don't know why the failure is not populated to the CI system. PR #1145 should fix this CI issue


          adding UT for frontend

6a54288

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

MSCetin37 force-pushed the docsum branch from b4621a5 to 6a54288 Compare

November 15, 2024 06:57

pre-commit-ci bot and others added 3 commits

November 15, 2024 06:57


          [pre-commit.ci] auto fixes from pre-commit.com hooks

d26c77f

for more information, see https://pre-commit.ci


          Merge branch 'main' into docsum

6663f9b


          removing UT for frontend

3a0b5bc

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

MSCetin37 force-pushed the docsum branch from 9a06cb9 to 3a0b5bc Compare

November 15, 2024 07:46


          removing UT for frontend

df1860e

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

MSCetin37 force-pushed the docsum branch from 0f7b165 to df1860e Compare

November 15, 2024 07:51

pre-commit-ci bot and others added 3 commits

November 15, 2024 07:52


          [pre-commit.ci] auto fixes from pre-commit.com hooks

a22439c

for more information, see https://pre-commit.ci


          Merge branch 'main' into docsum

f6decd1


          removing UT for frontend

750ea9c

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

lianhao suggested changes

View reviewed changes

DocSum/docker_image_build/build.yaml Outdated Show resolved Hide resolved

DocSum/docker_image_build/build.yaml Outdated Show resolved Hide resolved

DocSum/docker_image_build/build.yaml Outdated Show resolved Hide resolved

MSCetin37 and others added 2 commits

November 15, 2024 09:16


          Merge branch 'main' into docsum


          update the service names

e9939d1

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

dmsuehir reviewed

View reviewed changes

DocSum/docker_compose/intel/cpu/xeon/README.md Outdated Show resolved Hide resolved

DocSum/docker_compose/intel/cpu/xeon/README.md Outdated Show resolved Hide resolved

MSCetin37 added 2 commits

November 15, 2024 12:10


          update the readme

cd620bb

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          readme updates

c1f41c5

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

MSCetin37 force-pushed the docsum branch from 957a4e5 to c1f41c5 Compare

November 15, 2024 21:59


          readme updates

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

MSCetin37 force-pushed the docsum branch from c983c4c to 4448175 Compare

November 15, 2024 22:26

pre-commit-ci bot and others added 4 commits

November 15, 2024 22:27


          [pre-commit.ci] auto fixes from pre-commit.com hooks

bd7da2d

for more information, see https://pre-commit.ci


          readme updates

9eb5a26

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          readme updates

a6c9c2b

Signed-off-by: Mustafa <mustafa.cetin@intel.com>


          readme updates

5e87348

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

chensuyue reviewed

View reviewed changes

DocSum/docker_image_build/build.yaml Outdated Show resolved Hide resolved


          update build.yaml

46dc312

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

chensuyue reviewed

View reviewed changes

DocSum/docker_image_build/build.yaml Show resolved Hide resolved

chensuyue and others added 2 commits

November 17, 2024 12:46


          Merge branch 'main' into docsum

3ca23cb


          update build.yaml

1e99f0f

Signed-off-by: Mustafa <mustafa.cetin@intel.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

chensuyue chensuyue left review comments

dmsuehir dmsuehir left review comments

lianhao lianhao requested changes

ashahba ashahba approved these changes

lvliang-intel Awaiting requested review from lvliang-intel lvliang-intel is a code owner

WenjiaoYue Awaiting requested review from WenjiaoYue

eero-t Awaiting requested review from eero-t

XinyaoWa Awaiting requested review from XinyaoWa

At least 2 approving reviews are required to merge this pull request.

Labels

None yet