Skip to content

Move file processing from UI to DocSum backend service#1899

Merged
lvliang-intel merged 9 commits into
opea-project:mainfrom
mhbuehler:docsum-ui-multipart-form
May 8, 2025
Merged

Move file processing from UI to DocSum backend service#1899
lvliang-intel merged 9 commits into
opea-project:mainfrom
mhbuehler:docsum-ui-multipart-form

Conversation

@mhbuehler

Copy link
Copy Markdown
Collaborator

Description

Multipart form file uploads were only partially supported by the DocSum megaservice backend. This PR adds audio and video file support so that the API consumer does not have to convert files to base64 string format. Corresponding changes were made in DocSum's Gradio UI and in the example's READMEs showing the multipart form usage.

Issues

N/A

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Others (enhancement, documentation, validation, etc.)

Dependencies

None

Tests

Manual testing of the DocSum Gradio UI.

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
@mhbuehler mhbuehler requested a review from letonghan as a code owner May 1, 2025 22:53
@github-actions

github-actions Bot commented May 1, 2025

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Comment thread DocSum/docker_compose/amd/gpu/rocm/README.md
Comment thread DocSum/docker_compose/amd/gpu/rocm/README.md
Comment thread DocSum/docker_compose/amd/gpu/rocm/README.md
Comment thread DocSum/docker_compose/intel/cpu/xeon/README.md
mhbuehler added 2 commits May 5, 2025 14:52
Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>

@dmsuehir dmsuehir left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ashahba ashahba left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@MSCetin37 MSCetin37 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lvliang-intel lvliang-intel merged commit 7bb0558 into opea-project:main May 8, 2025
20 checks passed
yongfengdu pushed a commit to yongfengdu/GenAIExamples that referenced this pull request May 8, 2025
…1899)

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
@ashahba ashahba deleted the docsum-ui-multipart-form branch May 8, 2025 16:11
letonghan pushed a commit that referenced this pull request Sep 17, 2025
Signed-off-by: ZePan110 <ze.pan@intel.com>
cogniware-devops pushed a commit to Cogniware-Inc/GenAIExamples that referenced this pull request Dec 19, 2025
…1899)

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.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.

6 participants