GenAIExamples icon indicating copy to clipboard operation
GenAIExamples copied to clipboard

Published 20 hours ago • verified publisher icon opea-project

[DocSum] Aligned the output format

Open Zhenzhong1 opened this issue 6 months ago • 3 comments

Description

  • Aligned the output format

Issues

  • Fixed the issue. https://github.com/opea-project/GenAIComps/issues/1624

Type of change

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

  • [x] 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

List the newly introduced 3rd party dependency if exists.

Tests

curl -X POST http://10.7.4.144:8888/v1/docsum -H "Content-Type: application/json" -d '{"type": "text", "messages": "Text Embeddings Inference (TEI) is a toolkit for deploying and serving open source text embeddings and sequence classification models. TEI enables high-performance extraction for the most popular models, including FlagEmbedding, Ember, GTE and E5."}'

image

bash -x test_compose_on_gaudi.sh pass. image

Zhenzhong1 avatar May 14 '25 08:05 Zhenzhong1

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

github-actions[bot] avatar May 14 '25 08:05 github-actions[bot]

ready for merge. @chensuyue @lvliang-intel

Zhenzhong1 avatar May 20 '25 07:05 Zhenzhong1

ready for merge. @chensuyue @lvliang-intel

@Zhenzhong1 CI failed. Please check the CI issue.

xiguiw avatar May 22 '25 01:05 xiguiw

@Zhenzhong1 please check the Copilot comments.

chensuyue avatar May 30 '25 07:05 chensuyue

@Zhenzhong1 please check the Copilot comments.

Done, ready for merge.

Zhenzhong1 avatar Jun 03 '25 01:06 Zhenzhong1

DocSum Gaudi CI test timeouts when video input is beig tested. No idea whether that's due to added aligning functionality, or random test setup slowness.

eero-t avatar Jun 03 '25 09:06 eero-t

This PR is not the root cause of this CI test. It will be merged first.

Main branch CI test has the same error. https://github.com/opea-project/GenAIExamples/actions/runs/15481132796

There is a known issue to track this DocSum CI Fail case. opea-project/GenAIComps#1725.

Zhenzhong1 avatar Jun 06 '25 05:06 Zhenzhong1

This PR is not the root cause of this CI test. It will be merged first.

Main branch CI test has the same error. https://github.com/opea-project/GenAIExamples/actions/runs/15481132796

There is a known issue to track this DocSum CI Fail case. opea-project/GenAIComps#1725.

Ready for merge. @chensuyue

Zhenzhong1 avatar Jun 06 '25 05:06 Zhenzhong1