-
Notifications
You must be signed in to change notification settings - Fork 6.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
fix: Variable Aggregator Node supports parallel branching #8677
base: main
Are you sure you want to change the base?
Conversation
if "output" not in outputs: | ||
outputs["output"] = [] | ||
if isinstance(variable, list): | ||
outputs["output"].extend(variable) | ||
elif isinstance(variable, str): | ||
outputs["output"].append(variable) | ||
inputs[".".join(selector[1:])] = variable |
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.
Would you pls explain the purpose of this codes?
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.
When the parallel branch calls the knowledge recall and then aggregates, the variable is a list, and the parameter aggregation variable after the ordinary if else node is a str.
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.
When the parallel branch calls the knowledge recall and then aggregates, the variable is a list, and the parameter aggregation variable after the ordinary if else node is a str.
Could you explain the problem you’re encountering? I'd like to understand the context and intent.
If possible, pls create an issue and link it to this PR.
Checklist:
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsDescription
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue. Close issue syntax:
Fixes #<issue number>
, see documentation for more details.Fixes
Type of Change
Testing Instructions
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration