fix: deepcopy the inputs of components #6987
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issues
Proposed Changes:
Previously, when using the same input reference in different components, the Pipeline run logic had an unexpected behavior (see the original issue). This has been fixed by deepcopying the inputs before passing them to the components.
How did you test it?
CI, new test
Notes for the reviewer
data=deepcopy(data)
does not fix the issue.I think this happens because the same internal structure is reproduced, so if comp1 and comp2 share the same input reference, this also happens after deepcopy (comp1 and comp2 refer to the same deepcopied input)
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.