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

fix: Update message schema / data type to match OAI tools style #783

Merged
merged 7 commits into from
Jan 4, 2024

Conversation

cpacker
Copy link
Collaborator

@cpacker cpacker commented Jan 4, 2024

TODOs:

  • Make tool_calls JSON column
  • Modify db.py to match
  • Strip stray TODO comments
  • Make ToolCall record?

Please describe the purpose of this pull request.

Align new data types to be closer to OAI schema.

OAI:

image

@cpacker cpacker requested a review from sarahwooders January 4, 2024 20:11
@cpacker cpacker changed the title fix: Change message schema tools fix: Update message schema / data type to match OAI tools style Jan 4, 2024
@cpacker cpacker added the priority Merge ASAP label Jan 4, 2024
@cpacker cpacker force-pushed the change-message-schema-tools branch from 9cfc717 to c5e2648 Compare January 4, 2024 21:16
@cpacker cpacker changed the base branch from cherry-pick-storage-refactor to main January 4, 2024 21:16
Copy link
Collaborator

@sarahwooders sarahwooders left a comment

Choose a reason for hiding this comment

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

lgtm!

@cpacker cpacker merged commit febee38 into main Jan 4, 2024
3 checks passed
@cpacker cpacker deleted the change-message-schema-tools branch January 4, 2024 23:05
@sarahwooders sarahwooders restored the change-message-schema-tools branch January 4, 2024 23:08
@cpacker cpacker deleted the change-message-schema-tools branch January 12, 2024 07:06
norton120 pushed a commit to norton120/MemGPT that referenced this pull request Feb 15, 2024
mattzh72 pushed a commit that referenced this pull request Oct 9, 2024
Co-authored-by: Sarah Wooders <sarahwooders@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority Merge ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants