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

Fix for Issue #14663 #14668

Closed
wants to merge 9 commits into from
Closed

Conversation

gitchrisqueen
Copy link
Contributor

  • Description: Update to tools.py to allow aliased tool inputs to be included in the _parse_input function output
  • Issue: This fixes Issue #14663
  • Twitter handle: @consultchrisq

Need to set by_alias=True so that alias names will be included in the final results
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 13, 2023
Copy link

vercel bot commented Dec 13, 2023

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Dec 29, 2023 5:54pm

@dosubot dosubot bot added Ɑ: models Related to LLMs or chat model modules 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Dec 13, 2023
@baskaryan baskaryan assigned nfcampos and hinthornw and unassigned nfcampos Dec 13, 2023
@gitchrisqueen
Copy link
Contributor Author

@nfcampos @hinthornw - I see there are 2 failing checks. Is this something I should work to resolve or is that up to you and your team?

@hwchase17
Copy link
Contributor

not sure why, but its showing up as an empty PR. going to close, but feel free to reopen

@hwchase17 hwchase17 closed this Jan 2, 2024
@gitchrisqueen
Copy link
Contributor Author

PR from wrong forked branch. I created a new one here #15422

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature Ɑ: models Related to LLMs or chat model modules size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LangChain Agent bug when parsing tool inputs that use alias field names
4 participants