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

Davidmotson.auto functions #50

Closed
wants to merge 40 commits into from
Closed

Conversation

davidmotson
Copy link
Collaborator

DO NOT MERGE

David Motsonashvili added 2 commits January 30, 2024 14:36
includes a builder to make it easier to add functions
DO NOT MERGE
@davidmotson davidmotson requested a review from rlazo February 1, 2024 18:11
Copy link
Collaborator

@daymxn daymxn left a comment

Choose a reason for hiding this comment

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

Read through a bit of it, will read through the rest later

@rlazo
Copy link
Collaborator

rlazo commented Mar 20, 2024

We don't need to change the copyright date in the headers.

Copy link
Collaborator

@rlazo rlazo left a comment

Choose a reason for hiding this comment

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

Are there any changes necessary in common? all modifications are in generativeai

@@ -63,7 +73,19 @@ val GenerativeViewModelFactory = object : ViewModelProvider.Factory {
val generativeModel = GenerativeModel(
modelName = "gemini-1.0-pro",
apiKey = BuildConfig.apiKey,
generationConfig = config
generationConfig = config,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Given the inconsistent behaviour handling the functions, IMO we should have a separate entry in the sample app for function calling. This will prevent the regular chat example from failing due to an unrelated issue.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh whoops, this was not an intended part of the commit. I was doing this for my own testing, please ignore.

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.

5 participants