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

Simplify function calling #176

Merged
merged 10 commits into from
Jun 17, 2024
Merged

Simplify function calling #176

merged 10 commits into from
Jun 17, 2024

Conversation

daymxn
Copy link
Collaborator

@daymxn daymxn commented Jun 10, 2024

Per b/344929335,

This simplifies our function calling to not ascertain types. As a byproduct, we also no longer provide a means to execute functions. Additional documentation has been provided to showcase new usage.

This PR also fixes the following:

  • b/346341400 -> Provide a way to provide required params in function declaration

@daymxn daymxn requested a review from rlazo June 10, 2024 16:30
@daymxn daymxn self-assigned this Jun 10, 2024
@daymxn daymxn changed the title Daymon simplify funciton calling Simplify function calling Jun 10, 2024
@daymxn daymxn changed the title Simplify function calling Daymon simplify function calling Jun 10, 2024
@daymxn daymxn changed the title Daymon simplify function calling Simplify function calling Jun 10, 2024
@daymxn daymxn enabled auto-merge (squash) June 10, 2024 16:34
@rlazo
Copy link
Collaborator

rlazo commented Jun 10, 2024

Could you add a test? While the feature in itself doesn't have a lot logic to test, the test will also help demonstrate how to use the feature in its new incarnation.

@daymxn daymxn merged commit 12274f4 into main Jun 17, 2024
6 checks passed
@daymxn daymxn deleted the daymon-simplify-funciton-calling branch June 17, 2024 19:07
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
Per [b/344929335](https://b.corp.google.com/issues/344929335),

This simplifies our function calling to not ascertain types. As a
byproduct, we also no longer provide a means to execute functions.
Additional documentation has been provided to showcase new usage.
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
Per [b/344929335](https://b.corp.google.com/issues/344929335),

This simplifies our function calling to not ascertain types. As a
byproduct, we also no longer provide a means to execute functions.
Additional documentation has been provided to showcase new usage.
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.

2 participants