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

OpenAI-DotNet 8.1.2 #341

Merged

Conversation

henduck
Copy link
Contributor

@henduck henduck commented Jul 29, 2024

  • Added constructor overloads to Tool and Function classes to support manually adding tool calls in the conversation history

…the conversation history (setting ToolCallId, Function.Name, and Function.Arguments)
@henduck henduck requested a review from StephenHodgson as a code owner July 29, 2024 16:50
@StephenHodgson
Copy link
Member

I'll take a look shortly. Thanks for the contribution!

@jesses-avi
Copy link

Hi @StephenHodgson ...do you have a sense of when you might be able to look at this PR? I'm just trying to plan out my own timeline for addressing the use case this covers... and if it will be a long time before this is resolved, I'll just have to pursue an alternative path. I of course understand if this may not be your highest priority project, and appreciate your work on it in any case.

Thanks in advance!!

@StephenHodgson
Copy link
Member

StephenHodgson commented Aug 9, 2024

bump the minor version, update the change notes, and I'll run the unit tests and make a release if they pass

@henduck
Copy link
Contributor Author

henduck commented Aug 9, 2024

Awesome... done. Thanks!

Copy link
Member

@StephenHodgson StephenHodgson left a comment

Choose a reason for hiding this comment

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

we don't need to bump that far

@StephenHodgson StephenHodgson changed the title Support ToolCalls within the conversation history OpenAI-DotNet 8.1.2 Aug 9, 2024
@henduck henduck requested a review from StephenHodgson August 9, 2024 18:30
@StephenHodgson
Copy link
Member

All unit tests passed locally

@StephenHodgson StephenHodgson merged commit 8fcd8d8 into RageAgainstThePixel:main Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Not possible to instantiate a Tool with Tool.Id and Tool.Function.Arguments set?
3 participants