-
Notifications
You must be signed in to change notification settings - Fork 18k
community(you): Integrate You.com conversational APIs #23046
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
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.
these are cool! i think you need to merge master before CI can run
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.
Only adding unit tests for the .invoke
method.
I do not know how to use the requests_mock
library to mock a stream of bytes. I've attempted to use requests.post(..., content=content_callback)
where content_callback
returns a whole byte
string but the BaseLLM.stream
method raises an error. If I have force content_callback
to return a generator of byte
tokens, requests_mock
instead raises an error expecting content_callback
to return only one byte
string.
Checking in from You.com. We are hoping to rapidly increase and improve APIs. Let me know if we can support in any way. |
@hwchase17 how do we get this merged? |
Thanks @ccurme! |
You.com is releasing two new conversational APIs — Smart and Research.
This PR:
If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.