-
Notifications
You must be signed in to change notification settings - Fork 26
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
Move to HSVS parcellation #74
Comments
To include in this: Add command-line option |
Greetings Rob, the 5ttgen hsvs option has been integral to our present analysis :) Therefore, I am wondering if this has been integrated into the mrtrix3_connectome container as an option (I only see 5ttgen fsl in the code). If this would be relatively simple to implement, I can think about submitting a PR? Cheers, |
Hi Brian, No idea when I'll find time to do this; my many other obligations continue to take precedence. It wouldn't be the friendliest change to make. I've turned the code in this tool into a bit of a monster: in part I use it as an exemplar of the kind of analysis complexity that can be achieved using the MRtrix3 Python API---which includes being a single stand-alone file that can be executed outside of a container environment as long as you have all of the requisite dependencies---but this makes the code quite difficult to navigate. Am very much open to contributions; this is more of a pre-emptive "I told you so" :-P What would be needed is:
A preferable addition would be:
With code like this the hard part is making sure the code path is correct for all possible usages, rather than just your own specific usage. But if you want to give it a go, go for it, and I can review / amend. Cheers |
MRtrix3/mrtrix3#1935
Ideally provide a
-freesurfer
command-line option that can provide the location of FreeSurfer pre-generated outputs, e.g. fromfmriprep
, or runrecon_all
internally if necessary.The text was updated successfully, but these errors were encountered: