Skip to content
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

Updated examples/Voice_memos.ipynb to use google-genai SDK #593

Closed
wants to merge 1 commit into from

Conversation

prajwxlk
Copy link
Contributor

No description provided.

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:examples Issues/PR referencing examples folder labels Mar 22, 2025
@Giom-V Giom-V self-assigned this Mar 31, 2025
@@ -6,12 +6,12 @@
"id": "Tce3stUlHN0L"
Copy link
Collaborator

@Giom-V Giom-V Mar 31, 2025

Choose a reason for hiding this comment

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

Can you add a minimum version, like >=1.0.0?  


Reply via ReviewNB

@@ -6,12 +6,12 @@
"id": "Tce3stUlHN0L"
Copy link
Collaborator

@Giom-V Giom-V Mar 31, 2025

Choose a reason for hiding this comment

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

Line #4.        model="gemini-2.0-flash",

Can you add a "MODEL_ID" variable like this?

MODEL_ID="gemini-2.0-flash" # @param ["gemini-2.0-flash-lite","gemini-2.0-flash","gemini-2.0-pro-exp-02-05"] {"allow-input":true, isTemplate: true}

As it would make the notebook easier to maintain in the future.


Reply via ReviewNB

@Giom-V
Copy link
Collaborator

Giom-V commented Mar 31, 2025

Thanks @Chamikajaya for the submission and sorry for the delayed review.

Sadly that file was already being worked on in #589 and since the other PR is older I tend to give it priority (that's a rule that's as good as any other I guess).

That said, since feedback is the best way to improve, I just added a couple of minor comments to make the notebook easier to understand and to maintain.

I'll close the PR for now, but if for some reason #589 does not progress I'll reopen this one.

@Giom-V Giom-V closed this Mar 31, 2025
@prajwxlk
Copy link
Contributor Author

I'm really sorry Giom-V, I'll make sure to do a thorough check before creating a new PR. Thank you for the feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:examples Issues/PR referencing examples folder status:awaiting review PR awaiting review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants