Skip to content

Conversation

tonybaloney
Copy link
Contributor

functions is a deprecated argument. We should be using the tools parameter

@tonybaloney tonybaloney marked this pull request as ready for review January 24, 2024 02:02
@tonybaloney tonybaloney requested review from srbalakr and pamelafox and removed request for srbalakr January 24, 2024 02:03
@tonybaloney tonybaloney marked this pull request as draft January 24, 2024 03:06
@pamelafox
Copy link
Collaborator

I assume the OpenAI version we're supporting does support the new tools parameter?

@tonybaloney
Copy link
Contributor Author

I assume the OpenAI version we're supporting does support the new tools parameter?

Yes it does

…more typing was not the intention of this change
@tonybaloney tonybaloney marked this pull request as ready for review January 24, 2024 04:07
Copy link
Collaborator

@pamelafox pamelafox left a comment

Choose a reason for hiding this comment

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

Thank you!

@tonybaloney tonybaloney merged commit 62c5ae8 into Azure-Samples:main Jan 25, 2024
@tonybaloney tonybaloney deleted the functions_tools branch January 25, 2024 04:28
HughRunyan pushed a commit to RMI/RMI_chatbot that referenced this pull request Mar 26, 2024
* Update to tools parameter

* Use tool_choice argument

* Update response to iterate through new tool_calls property

* Allow tool_calls to be None

* Revert func signature as the local is implicitly optional and adding more typing was not the intention of this change

* Fix tests as the response structure looks different when using tool_calls
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.

3 participants