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.

Sign up for GitHub

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

Jump to bottom

OpenAI-DotNet 8.0.0 #309

Merged
merged 60 commits into from
Jun 10, 2024
Merged

OpenAI-DotNet 8.0.0 #309

merged 60 commits into from
Jun 10, 2024

Conversation

StephenHodgson
Copy link
Member

@StephenHodgson StephenHodgson commented May 19, 2024 •
edited
Loading

  • Updated Assistants Beta v2
  • Added support for specifying project id
  • Added BatchEndpoint
  • Added VectorStoresEndpoint
  • Added Message.ctr to specify specific tool call id, function name, and content
  • Renamed OpenAI.Images.ResponseFormat to OpenAI.Images.ImageResponseFormat
  • Changed ThreadEndpoint.CancelRunAsync return type from RunResponse to bool
  • Fixed Json defined Tools/Functions being improperly added to tool cache
  • Added Tool.TryUnregisterTool to remove a tool from the cache

TODO

@StephenHodgson
OpenAI-DotNet 8.0.0
e37694b 
- Updated Assistants Beta v2
@StephenHodgson StephenHodgson added this to the 8.0.0 milestone May 19, 2024
This was linked to issues May 19, 2024
Add ToolChoice to CreateRunRequest class #284
Closed
Assistants Beta V2 #285
Closed
Add Batch endpoint #287
Closed
Enable Assistant Tool Function Methods to be updated #299
Closed
JSON defined functions/tools should not be cached. #308
Closed
Add support for streaming with the assistants API #267
Closed
@StephenHodgson StephenHodgson self-assigned this May 19, 2024
@Stefan-Bechtel
Copy link

Great work! When will the merge be done?
Many thanks again!

@StephenHodgson
Copy link
Member Author

Great work! When will the merge be done? Many thanks again!

I just updated the description with todo task list.

@StephenHodgson StephenHodgson mentioned this pull request Jun 7, 2024
Merged
@leijae
Copy link

leijae commented Jun 10, 2024

... really looking forward to this update.

@StephenHodgson
Copy link
Member Author

... really looking forward to this update.

Trying to finish it up tonight!

@StephenHodgson
Copy link
Member Author

OK, just updating the docs and we're ready to rock and roll!

@StephenHodgson StephenHodgson marked this pull request as ready for review June 10, 2024 04:46
span
span approved these changes Jun 10, 2024
View reviewed changes
Copy link

@span span left a comment

Choose a reason for hiding this comment

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

Looks great. I have not had the opprtunity to test on real applications but can't see any issues. 👏

@StephenHodgson
don't cache json defined functions/tools
9d9216d 
added Tool.TryUnregisterTool to remove a tool from the cache
misc bug fixes and cleanup
@StephenHodgson
updated docs and final polish
cc452e0
@StephenHodgson StephenHodgson enabled auto-merge (squash) June 10, 2024 09:15
@StephenHodgson StephenHodgson merged commit 9675957 into main Jun 10, 2024
5 checks passed
@StephenHodgson StephenHodgson deleted the development branch June 10, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@span span span approved these changes

@hitchhiker hitchhiker Awaiting requested review from hitchhiker

Assignees

@StephenHodgson StephenHodgson

Labels
None yet
Milestone
8.0.0
6 participants
@StephenHodgson @default0 @Stefan-Bechtel @leijae @span @hitchhiker