-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
multiple[patch]: Switch deprecated model.call
to model.invoke
in tests and examples
#4726
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -43,6 +43,8 @@ docs/use_cases/question_answering/sources.md | |||
docs/use_cases/question_answering/sources.mdx |
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.
seems to have happened automatically - didn't manually change this
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.
Yeah, all good there
Fantastic, thank you! |
model.call
to model.invoke
model.call
to model.invoke
in tests and examples
model.call
to model.invoke
in tests and examplesmodel.call
to model.invoke
in tests and examples
Thank you! And congrats on your first PR! |
No-op.
Noticed that
model.call
was deprecated so updating the library and documentation tomode.invoke
Starter PR just to get the lay of the land
Ran
yarn test
and all tests continued to pass.