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

feat: Add support for struct. output #19

Merged
merged 11 commits into from
Nov 5, 2024

Conversation

gkaretka
Copy link
Contributor

  • add support for structured outputs
  • with function calling
  • with structured outputs

Copy link
Collaborator

@mrshu mrshu left a comment

Choose a reason for hiding this comment

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

Thanks @gkaretka, some small comments here and there

prompterator/main.py Outdated Show resolved Hide resolved
prompterator/main.py Outdated Show resolved Hide resolved
prompterator/utils.py Outdated Show resolved Hide resolved
@gkaretka gkaretka requested a review from mrshu October 24, 2024 21:40
- add support for structured outputs
- with function calling
- with structured outputs

Co-authored-by: Marek Šuppa <[email protected]>
@gkaretka gkaretka force-pushed the gkaretka/add-function-calling branch from df64fde to 896999e Compare October 25, 2024 12:23
Copy link
Collaborator

@mrshu mrshu left a comment

Choose a reason for hiding this comment

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

A couple more things @gkaretka, but looking good overall.

I'd still very much appreciate if @samsucik could take a look as well :)

prompterator/main.py Outdated Show resolved Hide resolved
prompterator/utils.py Outdated Show resolved Hide resolved
prompterator/utils.py Outdated Show resolved Hide resolved
@gkaretka gkaretka requested a review from mrshu October 29, 2024 13:09
Copy link
Collaborator

@samsucik samsucik left a comment

Choose a reason for hiding this comment

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

@gkaretka thanks a lot for this work. My only concerns are around code quality. This codebase is already quite messy, so let's try to not make it worse -- or else adding further changes will become more and more of a nightmare 😅

README.md Outdated Show resolved Hide resolved
prompterator/constants.py Outdated Show resolved Hide resolved
prompterator/constants.py Outdated Show resolved Hide resolved
prompterator/constants.py Show resolved Hide resolved
prompterator/main.py Outdated Show resolved Hide resolved
prompterator/main.py Show resolved Hide resolved
prompterator/utils.py Outdated Show resolved Hide resolved
prompterator/utils.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Show resolved Hide resolved
@gkaretka gkaretka force-pushed the gkaretka/add-function-calling branch from 655b9ef to dd05258 Compare October 31, 2024 21:41
- address PR comments

Co-authored-by: Sam Sucik <[email protected]>
Copy link
Collaborator

@samsucik samsucik left a comment

Choose a reason for hiding this comment

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

@gkaretka I've got only minor comments at the point. Once addressed (in whatever way you deem appropriate 😄), feel free to merge -- provided that everything works for you. Thanks 🚀

prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
prompterator/models/openai_models.py Outdated Show resolved Hide resolved
@gkaretka gkaretka force-pushed the gkaretka/add-function-calling branch from dd3ccfb to 9a102d2 Compare November 4, 2024 19:05
@samsucik samsucik merged commit 99a7eaf into slidoapp:main Nov 5, 2024
1 check passed
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