Skip to content
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

Harrison/error zero tools #6340

Merged
merged 2 commits into from
Jun 17, 2023
Merged

Harrison/error zero tools #6340

merged 2 commits into from
Jun 17, 2023

Conversation

hwchase17
Copy link
Contributor

No description provided.

juppytt and others added 2 commits June 17, 2023 09:43
<!--
Thank you for contributing to LangChain! Your PR will appear in our
release under the title you set. Please make sure it highlights your
valuable contribution.

Replace this with a description of the change, the issue it fixes (if
applicable), and relevant context. List any dependencies required for
this change.

After you're done, someone will review your PR. They may suggest
improvements. If no one reviews your PR within a few days, feel free to
@-mention the same people again, as notifications can get lost.

Finally, we'd love to show appreciation for your contribution - if you'd
like us to shout you out on Twitter, please also include your handle!
-->

Implemented validation for cases where no tools are provided. 
This change allows for the handling of scenarios where certain MRKL
tools, such as Zapier Natural Language Actions, may not support any
tools. With this validation, we prevent unnecessary chaining of API
usage.

This pull request is partially related to issue #5977.


#### Who can review?

@vowelparrot
@hwchase17 hwchase17 added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Jun 17, 2023
@vercel
Copy link

vercel bot commented Jun 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2023 4:54pm

@vercel vercel bot temporarily deployed to Preview June 17, 2023 16:54 Inactive
@hwchase17 hwchase17 merged commit 42a28ac into master Jun 17, 2023
@hwchase17 hwchase17 deleted the harrison/error-zero-tools branch June 17, 2023 18:00
This was referenced Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants