Skip to content

Conversation

@a9a4k
Copy link
Contributor

@a9a4k a9a4k commented May 17, 2025

Fixes #770

Tested with the same example as in the original issue and got event with 2 parts back correctly

@a9a4k a9a4k force-pushed the separate-thinking-and-text-parts-in-stream-mode branch from 184c6cf to 7996288 Compare May 18, 2025 20:08
@genquan9
Copy link
Contributor

The changes are verified and work well for latest codes.

@genquan9 genquan9 self-requested a review May 28, 2025 17:01
Copy link
Contributor

@genquan9 genquan9 left a comment

Choose a reason for hiding this comment

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

The changes are verified and work well.

@seanzhou1023 seanzhou1023 added ready to pull [Status] This PR is ready to be importing back to Google and removed ready to pull [Status] This PR is ready to be importing back to Google labels May 29, 2025
@seanzhou1023 seanzhou1023 added the ready to pull [Status] This PR is ready to be importing back to Google label May 29, 2025
copybara-service bot pushed a commit that referenced this pull request May 29, 2025
Copybara import of the project:

--
7996288 by Almas Akchabayev <almas.akchabayev@gmail.com>:

fix: separate thinking from text parts in streaming mode
COPYBARA_INTEGRATE_REVIEW=#777 from almeynman:separate-thinking-and-text-parts-in-stream-mode b63dcc7
PiperOrigin-RevId: 764561932
@seanzhou1023
Copy link
Collaborator

merged in commit 795605a

whhlct pushed a commit to whhlct/adk-python that referenced this pull request Jun 6, 2025
Copybara import of the project:

--
7996288 by Almas Akchabayev <almas.akchabayev@gmail.com>:

fix: separate thinking from text parts in streaming mode
COPYBARA_INTEGRATE_REVIEW=google#777 from almeynman:separate-thinking-and-text-parts-in-stream-mode b63dcc7
PiperOrigin-RevId: 764561932
coder-aditi pushed a commit to coder-aditi/adk-python that referenced this pull request Jul 2, 2025
Copybara import of the project:

--
7996288 by Almas Akchabayev <almas.akchabayev@gmail.com>:

fix: separate thinking from text parts in streaming mode
COPYBARA_INTEGRATE_REVIEW=google#777 from almeynman:separate-thinking-and-text-parts-in-stream-mode b63dcc7
PiperOrigin-RevId: 764561932
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to pull [Status] This PR is ready to be importing back to Google

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Thinking Parts and Text Parts get merged Into Single Part during Streaming Mode

3 participants