-
Notifications
You must be signed in to change notification settings - Fork 15
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
Workflow with separate M0 fails at registration step #319
Comments
@chidiugonna thank you for the detailed bug report. I won't be able to get to this until around Tuesday, but I'll try to get back to you then. |
@tsalo thank you! I just tried to update the filepath for |
@chidiugonna would you mind waiting a couple of hours until the |
Hi @tsalo ,
Here are some debugging outputs that show code flow - error is occurring on line 972:
ASL file has extension
Results from meanasl and meanm0 advertise
But in actual fact meanasl and meanm0 create files with extension
This causes flirt node to fail at line 972
|
Oof sorry. I've got a couple of ideas to fix it then. I'll probably just rely on the auto-generated filename, rather than passing one through the different nodes. |
I just merged #321 and I'm hoping it works. It'll be a couple of hours before the update pushes to DockerHub. |
@tsalo This works perfectly now! Thank you! |
That's a relief. Thank you for your patience with my debugging process. |
Summary
Aslprep crashes while executing Node "extract_deltam" with the following error message:
Additional details
What were you trying to do?
Ran Aslprep with the following command line call. My dataset included a PASL file and a separate M0 acquisition. The M0 file has 4 volumes while the PASL file (single PLD) has 20 volumes (control/label)
Error Log
Reproducing the bug
Error appears to happen in function
regmotoasl
in/usr/local/miniconda/lib/python3.8/site-packages/aslprep/interfaces/cbf.py
on line 966The FLIRT node is expecting
vol0000_xform-00000_merged_meanm0.nii
But the
meanm0
process actually createsvol0000_xform-00000_merged_meanm0.nii.gz
despite being explicitly set to produce the.nii
file on line 963.The text was updated successfully, but these errors were encountered: