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: Partial support for OpenAI-compatible assistant API #838

Merged
merged 30 commits into from
Feb 14, 2024

Conversation

sarahwooders
Copy link
Collaborator

@sarahwooders sarahwooders commented Jan 17, 2024

Please describe the purpose of this pull request.

  • Initial subset of function to allow users to interact with MemGPT agents in the same way as OpenAI's assistants API .
  • Modify server.user_message(...) to allow passing in a Message object (not just str).
  • Add an example using the OpenAI python client examples/openai_client_assistants.py
  • Add tests with the OpenAI python client and also python requests client

How to test
poetry run pytest -s tests/test_openai_assistant_api.py

Is your PR over 500 lines of code?
No

Todos:
Remaining todos in: #892

@sarahwooders sarahwooders marked this pull request as draft January 17, 2024 04:47
@cpacker cpacker changed the title [DRAFT] OpenAI-compatible assistant API feat: [DRAFT] OpenAI-compatible assistant API Jan 17, 2024
@Maximilian-Winter
Copy link
Contributor

@sarahwooders Would be happy to help with this one, I think this is a great addition. After paying 120$ dollars for Assistants API calls.

@sarahwooders
Copy link
Collaborator Author

@sarahwooders Would be happy to help with this one, I think this is a great addition. After paying 120$ dollars for Assistants API calls.

That's great to hear! I should be close to wrapping this up in the next 1-2 days but would be great to get your help with testing it out :)

@sarahwooders sarahwooders marked this pull request as ready for review January 22, 2024 05:21
@sarahwooders sarahwooders changed the title feat: [DRAFT] OpenAI-compatible assistant API Partial support for OpenAI-compatible assistant API Jan 22, 2024
@sarahwooders sarahwooders requested a review from cpacker January 22, 2024 22:39
@sarahwooders sarahwooders changed the title Partial support for OpenAI-compatible assistant API feat: Partial support for OpenAI-compatible assistant API Jan 22, 2024
@sarahwooders sarahwooders requested a review from cpacker February 13, 2024 23:15
Copy link
Collaborator

@cpacker cpacker left a comment

Choose a reason for hiding this comment

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

LGTM

@sarahwooders sarahwooders merged commit 391e01f into main Feb 14, 2024
4 checks passed
norton120 pushed a commit to norton120/MemGPT that referenced this pull request Feb 15, 2024
@cpacker cpacker deleted the assistant-api branch March 6, 2024 00:47
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.

4 participants