-
Notifications
You must be signed in to change notification settings - Fork 286
Add function call request param #79
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
Conversation
I would also update the readme to add function as feature |
get current weather
I am currently writing an example code for function calls, but I encountered an issue with the error: |
Hi @DaviRain-Su, Thank you for the PR and adding example for this new feature - its a great way to test it. Here are some hints for you to debug this further:
Hope this helps |
Successfully added an example that can use function calls. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears that naming does not adheres to OpenAPI spec.
For example ChatCompletionFunctions
, Please see the names of new types in this PR #73 - that's the unfortunate part of having types hand written instead of having it auto generated. That said, naming should be consistent with rest of the project
For background: all/majority of types in types.rs
are named based on OpenAPI component schema.
…ontrolling how the model responds to function calls, and adds options for generating multiple completions.
Have fix all. |
Thank you for addressing the comments, nice work on the new example! |
This is released in v0.12.0 |
close: #72
#76