Skip to content

Feature: Added a demo for learnlm model#744

Merged
Giom-V merged 16 commits intogoogle-gemini:mainfrom
andycandy:learnlm
Jun 4, 2025
Merged

Feature: Added a demo for learnlm model#744
Giom-V merged 16 commits intogoogle-gemini:mainfrom
andycandy:learnlm

Conversation

@andycandy
Copy link
Copy Markdown
Collaborator

@andycandy andycandy commented Apr 28, 2025

Closes #732

  • Added Get_Started_Learnlm notebook in quickstarts folder
  • nbfmt and lint checked

@review-notebook-app
Copy link
Copy Markdown

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 28, 2025
@andycandy
Copy link
Copy Markdown
Collaborator Author

@Giom-V @markmcd

Comment thread quickstarts/Get_Started_Learnlm.ipynb
Comment thread quickstarts/Get_Started_Learnlm.ipynb
Comment thread quickstarts/Get_Started_Learnlm.ipynb
Comment thread quickstarts/Get_Started_Learnlm.ipynb
Comment thread quickstarts/Get_Started_Learnlm.ipynb
@nikitamaia
Copy link
Copy Markdown
Collaborator

@andycandy this is great! I added a few minor comments to clarify some of the markdown instructions.

@andycandy
Copy link
Copy Markdown
Collaborator Author

@nikitamaia thank you for the feedback, I'll work on those suggestions now

nikitamaia
nikitamaia previously approved these changes May 2, 2025
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

@Giom-V Giom-V left a comment

Choose a reason for hiding this comment

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

@andycandy I was about to merge the PR, but saw some minor stuff that I won't be easily able to change in a subsequent CL:

  • Can you rename the file "Get_started_LearnLM.ipynb to stay consistent with the other get started guides? That was the blocking one
  • Can you copy and adapt to yourself the contributor badge I'm introducing in https://github.com/Giom-V/gemini-api-cookbook/blob/badges/examples/Virtual_Try_On.ipynb ?
  • While you're at it, I would also move the "Help me study for a undergrad cognition test on theories of emotion generation" prompt to the cell below where it is.

@Giom-V
Copy link
Copy Markdown
Collaborator

Giom-V commented May 6, 2025

@andycandy You also need to update the quickstarts readme to add a link to the notebook (maybe not the main one as it's not a new model)

@andycandy
Copy link
Copy Markdown
Collaborator Author

On it @Giom-V!

@andycandy
Copy link
Copy Markdown
Collaborator Author

Done!

@andycandy andycandy requested a review from Giom-V May 6, 2025 15:14
Giom-V
Giom-V previously approved these changes May 6, 2025
@andycandy andycandy requested a review from Giom-V May 6, 2025 21:23
@Giom-V Giom-V merged commit 2eb2aed into google-gemini:main Jun 4, 2025
5 checks passed
@andycandy andycandy deleted the learnlm branch June 4, 2025 19:32
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.

Feature Request: Add Notebook Examples for LearnLM

3 participants