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

Add responseMimeType support #133

Merged
merged 9 commits into from
May 7, 2024
Merged

Conversation

daymxn
Copy link
Collaborator

@daymxn daymxn commented May 2, 2024

Per b/338414499,

This adds support for the responseMimeType property that was added to GenerationConfig of gemini.

This PR also fixes the following:

DO_NOT_MERGE -> Wait until idea iteration is complete

@daymxn daymxn requested a review from rlazo May 2, 2024 17:10
@daymxn daymxn self-assigned this May 2, 2024
@rlazo
Copy link
Collaborator

rlazo commented May 2, 2024

Please add tests

@daymxn daymxn requested a review from rlazo May 3, 2024 22:11
@rlazo rlazo merged commit 14c0ba6 into main May 7, 2024
5 checks passed
@rlazo rlazo deleted the daymon-add-responsemimetype-support branch May 7, 2024 03:04
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
Per [b/338414499](https://b.corp.google.com/issues/338414499),

This adds support for the `responseMimeType` property that was added to
`GenerationConfig` of gemini.

This PR also fixes the following:
- [b/338415129](https://b.corp.google.com/issues/338415129) -> Hide
leaked JSON property in common

---------

Co-authored-by: Rodrigo Lazo <[email protected]>
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
Per [b/338414499](https://b.corp.google.com/issues/338414499),

This adds support for the `responseMimeType` property that was added to
`GenerationConfig` of gemini.

This PR also fixes the following:
- [b/338415129](https://b.corp.google.com/issues/338415129) -> Hide
leaked JSON property in common

---------

Co-authored-by: Rodrigo Lazo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants