Skip to content
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

ENH: CiftiDilate interface #50

Merged
merged 2 commits into from
Apr 30, 2021
Merged

ENH: CiftiDilate interface #50

merged 2 commits into from
Apr 30, 2021

Conversation

mgxd
Copy link
Collaborator

@mgxd mgxd commented Apr 29, 2021

Preparation for #49

Currently untested.

@mgxd mgxd self-assigned this Apr 29, 2021
@mgxd
Copy link
Collaborator Author

mgxd commented Apr 30, 2021

Testing this with a dtseries produced by fMRIPrep. The command mirrors the one in the DCAN script

wb_command -cifti-dilate sub-01_task-mixedgamblestask_run-1_space-fsLR_den-91k_bold.dtseries.nii COLUMN 0 10 dilated_sub-01_task-mixedgamblestask_run-1_space-fsLR_den-91k_bold.dtseries.nii

But I'm running into

ERROR: left surface required but not provided

@effigies
Copy link
Member

I wonder if there's a wb_spec dependency that wasn't listed in the script.

@mgxd
Copy link
Collaborator Author

mgxd commented Apr 30, 2021

After chatting with @ericfeczko it seems their CIFTI is purely volumetric at that step, so no surface information is required.

@effigies
Copy link
Member

Ah, yeah, it's running on something made at L107.

@mgxd
Copy link
Collaborator Author

mgxd commented Apr 30, 2021

@effigies I'm going to merge this in so we can then just append new interfaces to nibabies.interfaces.workbench.py.

@mgxd mgxd merged commit e78aa4b into nipreps:master Apr 30, 2021
@mgxd mgxd deleted the enh/cifti-dilate branch April 30, 2021 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants