-
Notifications
You must be signed in to change notification settings - Fork 14
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
single modality at a time #137
Conversation
- change modality from a list to a string - removes modality wildcard from all (except work tar file) - changes seg_{modality} and surf_{modality} to seg_ and surf_ (allowing for further refactoring) - T2w workflow updated and dry-run tested, others are WIP
ok this looks good to review and merge in -- I didn't do any other refactoring here (except for renaming |
This might be a good place for one more thing - perhaps we should change the default modality to |
Yeah I agree, I think defaulting to T2w doesn't make sense anymore.. I am thinking we should just make |
Ah didn't realize you updates the name. |
ya I updated those already with a recent merge of master.. I'll fix the remaining conflicts now (after merging in neonatal) then merge this in |
for further refactoring)