-
Notifications
You must be signed in to change notification settings - Fork 54
ENH: Workflow splicer module #938
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #938 +/- ##
==========================================
+ Coverage 71.12% 71.45% +0.33%
==========================================
Files 87 89 +2
Lines 8477 8618 +141
Branches 1057 1072 +15
==========================================
+ Hits 6029 6158 +129
- Misses 2220 2227 +7
- Partials 228 233 +5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Trying to wrap my head around this. I left a couple of comments.
1.13.1 (May 02, 2025) Patch release in the 1.13.x series. This release adds the ``niworkflows.engine.splicer`` module, to facilitate sugery on wworkflows. * FIX: Allow passing arguments through tag decorator (#939) * ENH: Workflow splicer module (#938) * MNT: Bump astral-sh/setup-uv from 5 to 6 (#937)
This PR lays some of the groundwork for fMRIPrep extensions.
splice_workflowsplices a constructed workflow, allowing developers to swap in alternative processing workflows. This works by finding all taggedNode/Workflowin a graph, and substituting the matchingreplacementsworkflow. This will require some work in the NiPreps to encapsulate each processing step, likely with a heavy dosing of buffernodes for input and output fields.Additionally, the
@tagdecorator can be used on anyinit_...wf()factory to facilitate adding the_tagattribute.