-
Notifications
You must be signed in to change notification settings - Fork 121
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 structured output #112
Conversation
Signed-off-by: Sternakt <sternakt@gmail.com>
We are waiting for @sternakt to push his changes to this branch. |
Signed-off-by: Sternakt <sternakt@gmail.com>
@ohdearquant could you help review this? Please approve if it reads good to you! Thank you! |
@davorrunje @sternakt It seems meta data of the notebook included in this PR was not updated properly. I made direct changes to the notebook, so we are all good here. Please remember to check meta data in future notebooks. Thank you! |
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.
Looks good. Main changes are the response format and oai client, which is a clean straightforward implementation.
Why are these changes needed?
Add
structured_format
parameter to theConversibleAgent
class and use it to define JSON output format of the agent.Related issue number
Closes #23
Checks