Skip to content

Conversation

vinupreethi
Copy link

Thank you for taking time to contribute this pull request!
You might have already read the contributor guide, but as a reminder, please make sure to:

  • Add a Signed-off-by line to each commit (git commit -s) per the DCO
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

For more details, please check the contributor guide.
Thank you upfront!

Signed-off-by: vinothini <vinupreethi@users.noreply.github.com>
@markpollack
Copy link
Member

markpollack commented Sep 28, 2025

@vinupreethi this is a large pr to review just from the source code changes. can you explain more please?

@vinupreethi
Copy link
Author

@markpollack i was working at this issue... #3816 (comment) I'm a newbie..

@markpollack
Copy link
Member

Thanks @vinupreethi so is this PR still relevant?

@markpollack markpollack added bug Something isn't working status: to-discuss labels Sep 28, 2025
@vinupreethi
Copy link
Author

vinupreethi commented Sep 28, 2025

@markpollack @192902649 need to verify the changes., When using Flux, the context is returned with every response, which results in large data volume. This can be optimized — for example, only include the context in the response when chatResponse.result.metadata.finishReason == "stop". this is the description given.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status: to-discuss
2 participants