Tool Insights #242
kichanyurd
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Motivation
This discussion is based on issues raised in #237 by @ElisonSherton. Primarily:
Consider the following use case:
When the user needs to make a transaction; Then assist them in the process
connected to a tooltransfer_money(destination_account: str, amount: float, payment_date: str)
.Hey, I'd like to send $200 to John's account
payment_date
is a required parameter, the tool cannot be called. TheToolCaller
understands this and therefore does not attempt to calltransfer_money()
.MessageEventGenerator
has no way to know that theToolCaller
was only missing one argument to make the call.MessageEventGenerator
, not seeing a success indicator from a tool call event, would still try to comply with the its guideline toassist them in the process (of making a transaction)
.I would love to help you with this transaction. Let me see what I can do and get back to you.
or something fairly useless of that nature.MessageEventGenerator
is not to blame here. It just lacks information, which theToolCaller
has; namely, that it would have liked to run thetransfer_money()
tool — it just couldn't becausepayment_date
was missing.Solution Expectations
ToolCaller
's evaluation into theMessageEventGenerator
's context, making it accessible for its consideration.datetime
. We currently only support basic types likestr
,int
, etc.Solution Proposal
Part 1: Communicating Tool Insights to the
MessageEventGenerator
TBD
Part 2: Supporting Additional Parameter Types
TBD
Part 3: Improving the Ability to Annotate Parameters
TBD
Discussion
I will update the Solution Proposal soon enough. I'd like to first get some input on the above and see if anyone has additional interesting use cases for consideration here.
Beta Was this translation helpful? Give feedback.
All reactions