Skip to content

Enable remote inference non streaming response #1806

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 8 commits into from
Jul 2, 2025

Conversation

srinarayan-srikanthan
Copy link
Collaborator

Description

Enable remote inference non streaming response

Issues

Na

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

na

Tests

na

Signed-off-by: Ubuntu <azureuser@denvr-inf.kifxisxbiwme5gt4kkwqsfdjuh.dx.internal.cloudapp.net>
Copy link

codecov bot commented Jun 20, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
comps/cores/mega/micro_service.py 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
comps/cores/mega/orchestrator.py 90.43% <100.00%> (ø)
comps/cores/mega/micro_service.py 79.43% <0.00%> (+0.73%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

srinarayan-srikanthan and others added 5 commits June 20, 2025 09:43
Signed-off-by: Ubuntu <azureuser@denvr-inf.kifxisxbiwme5gt4kkwqsfdjuh.dx.internal.cloudapp.net>
Signed-off-by: Ubuntu <azureuser@denvr-inf.kifxisxbiwme5gt4kkwqsfdjuh.dx.internal.cloudapp.net>
Signed-off-by: Ubuntu <azureuser@denvr-inf.kifxisxbiwme5gt4kkwqsfdjuh.dx.internal.cloudapp.net>
Copy link
Collaborator

@letonghan letonghan left a comment

Choose a reason for hiding this comment

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

LGTM, please check the failed CI test, thanks.

@srinarayan-srikanthan
Copy link
Collaborator Author

srinarayan-srikanthan commented Jul 1, 2025

@letonghan @louie-tsai @chensuyue and @lvliang-intel , need help with this PR to be merged. The test is failing, timing out after 1hr. Not sure what the issue is. Not related to the changes in this PR.

@chensuyue
Copy link
Collaborator

@letonghan @louie-tsai @chensuyue and @lvliang-intel , need help with this PR to be merged. The test is failing, timing out after 1hr. Not sure what the issue is. Not related to the changes in this PR.

Yes, the CI issue is known issue, #1725

@srinarayan-srikanthan
Copy link
Collaborator Author

So can this PR be merged, two other PRs are dependent on this. @chensuyue @lvliang-intel @ftian1 @letonghan

@chensuyue chensuyue merged commit 51f106f into opea-project:main Jul 2, 2025
23 of 25 checks passed
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.

5 participants