8000 Replace propagate_rerun with being able to pass list of node names to rerun · Issue #786 · nipype/pydra · GitHub
[go: up one dir, main page]
Skip to content

Replace propagate_rerun with being able to pass list of node names to rerun #786

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.

Sign up for GitHub

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

Jump to bottom
Open
tclose opened this issue Apr 3, 2025 · 1 comment
Open

Replace propagate_rerun with being able to pass list of node names to rerun #786

tclose opened this issue Apr 3, 2025 · 1 comment
Assignees
@tclose
Labels
feature Feature requests

Comments

@tclose
Copy link
Contributor
tclose commented Apr 3, 2025 •
edited
Loading

This will allow much finer grain control over which nodes of a workflow need to be run than propagate_rerun, which is kind of useless as if you set it to False because re-runing identical workflows but not their nodes is guaranteed not to change anything.

my_workflow = MyWorkflow(in_file='/path/to/nifti')

outputs = my_workflow(rerun=["data_source"])

and for nested workflow nodes you could specify using '.'

outputs = my_workflow(rerun=["nested_wf.stochastic_alg", "nested_wf2.add_noise"])

In the process, it would be nice to make rerun part of the Job object so we can drop rerun from the Task._run(job: Job[Task], rerun: bool = False) signature to make it a little cleaner
@tclose tclose self-assigned this Apr 3, 2025
@tclose tclose converted this from a draft issue Apr 3, 2025
@tclose tclose added enhancement New feature or request feature Feature requests and removed enhancement New feature or request labels Apr 29, 2025
@tclose tclose moved this from Proposed to temp in Pydra Roadmap Apr 29, 2025
@tclose tclose moved this from temp to To do (v1.0) in Pydra Roadmap Apr 29, 2025
@effigies
Copy link
Contributor

I like this, but I don't think it needs to be 1.0.

@tclose tclose moved this from Triage to v1.0 in Pydra Roadmap May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tclose tclose

Labels
feature Feature requests
Projects
Pydra Roadmap
Status: v1.0
Milestone
No milestone
Development

No branches or pull requests
2 participants
@effigies @tclose
0