-
Notifications
You must be signed in to change notification settings - Fork 161
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
Enable NonlinearGradientCorrection to take values "2D" and "3D" #1160
Comments
I don't think this is needed or even recommended because 2D or 3D is what we know Siemens encodes, but we do not know other manufacturers - also other algorithms are possible. The way we trach this so far is in the name with '_rec-xxxx' for instance '_rec-normdist2Dcorr'. Also, in the name with '_rec-normdistcorr' when in a derivative folder with in the json we add "wasGeneratedBy": "gradient_nonlin_unwarp" (the code we use to compute our own version) |
Most of the data I have seen have |
both corrections cannot be applied sequentially - our scanner does not let people do that -- what happens is your data are corrupted, I guess (for instance, one could use the manual correction on top of Siemens one) |
In the Inline card on the console, it is set to 3D correction, but it results in both DIS2D and DIS3D in ImageType, and it is not manual operation. See https://github.com/courtois-neuromod/anat/tree/main/sub-01/ses-001/anat for all the files with I don't think this is specific to our scanner though, I have seen this in other VE-series data, and even with recent XA30 data from another site (Prisma Fit too), the ImageTypeText has the same, maybe misleading, info. Is it not the case for your Siemens data with 3D correction? I searched all the QA datasets of dcm2niix, and none seems to feature |
our Prisma only spits out |
Siemens reports nonlinear gradient correction with DIS2D, DIS3D, and ND (via the syngo MR E11E conformance statement:
BIDS has a simple boolean option:
For backwards compatibility, I would suggest that we map:
false
"2D"
"3D"
I propose keeping
false
instead of adding a new string like"None"
, so that usingbool(meta["NonlinearGradientCorrection"])
will continue to work as expected.true
needs to remain valid for backwards compatibility, but tools will be unable to identify it as either 2D or 3D.Any objections in principle? If not I can propose an update to the schema that will permit any of these values.
via rordenlab/dcm2niix#597 (comment)
The text was updated successfully, but these errors were encountered: