-
Notifications
You must be signed in to change notification settings - Fork 430
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
Adds support for the Assistants API #192
base: master
Are you sure you want to change the base?
Conversation
Excited for this one to be merged as it is needed for reading uploaded files. @georgegkonis how has it been working for you? |
When shall we expect the merge of this pull request? Is there a work around dealing with assistants and files for now? Thank you! |
I haven't gotten any feedback from the author and it's been months since I've opened the PR so I'd say chances of it being merged are slim to none. From a quick look, I see there's no activity on all new issues, so maybe it is that the project isn't being actively worked on for the time being. I've since migrated to Azure's library for my needs, as new features are being implemented much faster. It's a shame, as I much preferred the approach this solution took, but it is what it is. To answer your question, @Tronald, the solution was working well for me, though I didn't have the opportunity to test it extensively across multiple scenarios—just the specific ones I needed. |
@georgegkonis thanks for the heads up. I really like this library, but as with many open source projects, there comes the risk that those running it have to attend to higher priorities. I suppose I will look into the Azure solutions as well, though they seem a bit convoluted in comparison sadly. |
@georgegkonis looks like this commit supports Assistant v1, would it work with v2 as well? Edit: Doesn't look like it would. This commit still uses v1 objects, and doesn't fit the descriptions of this Migration Guide: |
Just an FYI everyone. I know @OkGoDoIt can't talk about the big news, but based on the beta package posted to Nuget last week, and the below fork, I would say we gambled correctly in choosing this library! Looks like it's about to be the official OpenAI .NET library. This is purely speculation based on the open evidence I stumbled upon, but if true, then well done @OkGoDoIt.... https://www.nuget.org/packages/OpenAI/#versions-body-tab https://github.com/openai/openai-dotnet/tree/OpenAI_2.0.0-beta.3 https://github.com/openai/openai-dotnet/blob/main/MigrationGuide.md |
Added support for the Assistants API.
Tried to integrate it as best as I could without making any changes to the other parts of the code. Still WiP as I haven't had the time to write any tests yet, and a couple result classes are missing and are simply "object" for now. Open to feedback.