-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
add option to send user message as part of request body in assistant … #1322
base: main
Are you sure you want to change the base?
Conversation
@nsarrazin please let me know your thoughts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! Thanks for the contrib!
Do you think we could instead do something like {{url|post=https://example.com}}
so that we can keep the old functionality as well ?
@nsarrazin about keeping the old functionality: could you approve the workflow again? I had to resolve some merge conflicts. |
Yep, my point was that if we use a different keyword like |
Yes, that actually makes more sense. I’ll do that. |
Hi @ethayu do you still want to take a look at this? Just curious no pressure 😄 |
@nsarrazin sorry, I've been a little swamped with school recently! I'll get to it around thanksgiving break, if nobody else has taken care of it by then |
Add support for to send user message/query within assistant invocations, as referenced in #1302.