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

Adds support for the Assistants API #192

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

georgegkonis
Copy link

@georgegkonis georgegkonis commented Feb 4, 2024

Added support for the Assistants API.

  • Added Assistants endpoint
  • Added Threads endpoint
  • Added Messages endpoint
  • Added Runs endpoint

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.

@georgegkonis georgegkonis marked this pull request as ready for review February 5, 2024 09:55
@Tronald
Copy link

Tronald commented Apr 11, 2024

Excited for this one to be merged as it is needed for reading uploaded files. @georgegkonis how has it been working for you?

@Abdesol
Copy link

Abdesol commented May 6, 2024

When shall we expect the merge of this pull request? Is there a work around dealing with assistants and files for now? Thank you!

@georgegkonis
Copy link
Author

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.

@Tronald
Copy link

Tronald commented May 8, 2024

@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.

@tangrui1992
Copy link

tangrui1992 commented May 19, 2024

@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:
https://platform.openai.com/docs/assistants/migration

@Tronald
Copy link

Tronald commented Jun 10, 2024

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

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