-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Conversation
@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 :) |
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.
LGTM
Please describe the purpose of this pull request.
server.user_message(...)
to allow passing in aMessage
object (not juststr
).examples/openai_client_assistants.py
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