-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
refactor(FlowTool.py): update FlowToolComponent class to inherit from LCToolComponent #3048
Conversation
…_result_data function for better readability and maintainability
…om LCToolComponent
Pull Request Validation ReportThis comment is automatically generated by Conventional PR Whitelist Report
Result Pull request does not satisfy any enabled whitelist criteria. Pull request will be validated. Validation Report
Result Pull request is invalid. Reason
Last Modified at 29 Jul 24 22:50 UTC |
This pull request is automatically being deployed by Amplify Hosting (learn more). |
The Dropdown is also allowing the user to select multiple flows (and none of them appears as selected in the field) https://jam.dev/c/6ad09f07-a670-4471-8b98-c0814f515617 And another bug just ocurred after this. I switched back to the Langflow tab, got an "Unexpected Error", and the component showed a Warning sign to update it. After updating it, the dropdown to select flows stopped working. https://jam.dev/c/161f2b66-80b3-4fab-964b-c1474f39a911 That was wierd bc I didn't touch the backend, so there was no update or pull, between those two videos. |
… LCToolComponent (langflow-ai#3048) * refactor(utils.py): simplify data processing logic in build_data_from_result_data function for better readability and maintainability * feat: set default output type to "chat" in run_flow function * refactor(FlowTool.py): refactor FlowToolComponent class to inherit from LCToolComponent * Fixed Flow as Tool component * Fixed FlowTool to only output tool and removed async function def * [autofix.ci] apply automated fixes * Fixed lint --------- Co-authored-by: Lucas Oliveira <62335616+lucaseduoli@users.noreply.github.com> Co-authored-by: Lucas Oliveira <lucas.edu.oli@hotmail.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> (cherry picked from commit 8bc45b9)
Updates FlowToolComponent to the new Component architecture.