Skip to content

Feat: Documenting 2.5 Flash thinking in REST/curl #729

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 11 commits into from
Jul 31, 2025

Conversation

andycandy
Copy link
Collaborator

Closes #723

Feat:

  • Adds a notebook for showcasing Gemini 2.5 flash thinking
  • nbfmt and lint checked

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added status:awaiting review PR awaiting review from a maintainer component:quickstarts Issues/PR referencing quickstarts folder labels Apr 20, 2025
Copy link

review-notebook-app bot commented May 1, 2025

View / edit / reply to this conversation on ReviewNB

nikitamaia commented on 2025-05-01T21:49:14Z
----------------------------------------------------------------

empty link. maybe worth removing since flash 2.0 is not used in this notebook.


andycandy commented on 2025-05-01T22:29:35Z
----------------------------------------------------------------

Alright I removed that link.

Copy link

review-notebook-app bot commented May 1, 2025

View / edit / reply to this conversation on ReviewNB

nikitamaia commented on 2025-05-01T21:49:15Z
----------------------------------------------------------------

None?


andycandy commented on 2025-05-01T22:14:50Z
----------------------------------------------------------------

Technically the response doesn't contain any thoughts tokens because it didn't use any, so while trying to access that object it throws an error so we are using a placeholder which shows up when there is no thoughts tokens included in the metadata.

Copy link

review-notebook-app bot commented May 1, 2025

View / edit / reply to this conversation on ReviewNB

nikitamaia commented on 2025-05-01T21:49:15Z
----------------------------------------------------------------

Here you say "you are fixing a value to the thinking budget" but in the payload below its set to 0. Can you clarify this?


andycandy commented on 2025-05-01T22:24:20Z
----------------------------------------------------------------

Apologies, it was a typo

@nikitamaia nikitamaia self-requested a review May 1, 2025 21:50
Copy link
Collaborator Author

Apologies, it was a typo


View entire conversation on ReviewNB

Copy link
Collaborator Author

Alright I removed that link.


View entire conversation on ReviewNB

nikitamaia
nikitamaia previously approved these changes May 2, 2025
Copy link
Collaborator

@nikitamaia nikitamaia left a comment

Choose a reason for hiding this comment

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

lgtm! thank you!

Copy link

review-notebook-app bot commented May 2, 2025

View / edit / reply to this conversation on ReviewNB

nikitamaia commented on 2025-05-02T15:52:50Z
----------------------------------------------------------------

actually one more thing, seems like this links back to the non rest example.


andycandy commented on 2025-05-02T16:27:43Z
----------------------------------------------------------------

Oh yeah, lemme fix that real quick.

andycandy commented on 2025-05-02T16:33:55Z
----------------------------------------------------------------

Fixed that!

Copy link
Collaborator Author

Oh yeah, lemme fix that real quick.


View entire conversation on ReviewNB

Copy link
Collaborator Author

Fixed that!


View entire conversation on ReviewNB

@andycandy andycandy requested a review from nikitamaia May 3, 2025 10:00
GladiatorIR

This comment was marked as spam.

@Giom-V Giom-V merged commit 2516210 into google-gemini:main Jul 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:quickstarts Issues/PR referencing quickstarts folder status:awaiting review PR awaiting review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document 2.5 Flash thinking in REST/curl examples
4 participants