-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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