-
Notifications
You must be signed in to change notification settings - Fork 75
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
[REF] Extract Step 2 of DWIPreprocessingUsingPhasediff
into its own workflow
#897
[REF] Extract Step 2 of DWIPreprocessingUsingPhasediff
into its own workflow
#897
Conversation
36f9ece
to
f7551bd
Compare
The Jenkins failure is expected as the data needed for the tests aren't merged and uploaded yet. |
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.
Looks good. Just one small suggestion.
I'll test it asap with the data you've generated.
...a/pipelines/dwi_preprocessing_using_fmap/dwi_preprocessing_using_phasediff_fmap_workflows.py
Show resolved
Hide resolved
f7551bd
to
ef3c471
Compare
Tests are passing with the updated data. |
Co-authored-by: Matthieu Joulot <85217698+MatthieuJoulot@users.noreply.github.com>
Thanks for the reviews @MatthieuJoulot ! |
Similar to #893 for Step 1, this PR proposes to extract the Step 2 of the
DWIPreprocessingUsingPhaseDiff
pipeline into its own workflow:calibrate_and_register_fmap
This PR also proposes to add functional tests for the extracted workflow in isolation.
The added test is classified as "fast" as it is running in about 1 minute. It requires the data I have generated here: https://github.com/aramis-lab/clinica_data_ci/pull/29