-
Notifications
You must be signed in to change notification settings - Fork 7
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
DWI directions utilised in model #48
Comments
@Lestropie I wonder whether we should try to cover 80-90 percent of the current data and software (for that gradients would work I think) or be more forward-looking and discuss more solutions for future data. |
I suspect the functional coverage of " Utilising only a subset of the diffusion directions is itself uncommon, so does it even need to be part of the spec? I can see that " If the gradient table were already in the IJKb / XYZb format rather than |
Working on #92 highlighted that this was potentially causing more problems than it solved:
So I'm going to close this off as having been removed from the specification. If there's adequate motivation for having the capacity to track such provenance, then it will need more dedicated consideration. |
The more general solution here is to store the gradient table of the data to which the model was fitted. Applies to non-shelled data, applies to use of a subset of data, and will eventually apply to the case of advanced DWI acquisitions. We already do something akin to this in MRtrix3 MRtrix3/mrtrix3#774.
The text was updated successfully, but these errors were encountered: