-
Notifications
You must be signed in to change notification settings - Fork 171
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
Unknown Contrast Agent in BIDS #1542
Comments
I don't think we have had to face the issue yet. So this is a first one. I tested on one of the bids example with some dummy values (see below) and it seems that the validator does not catch those issues... Yet. Once the validator starts on the schema, those may get flagged as errors
$ cat ds001/sub-01/anat/sub-01_ce-foo_T1w.json
{"ContrastBolusIngredient": "foo"} |
Thanks for you answer! |
Maybe adding an "unknown" case in the specification would make sense here. I definitely would not be opposed to it. @effigies what do you think? |
Seems like there's a legitimate use case, so we should support it. Also seems like there should be a |
Thanks for your help, including these options in the specification would be great! |
This is what the new metadata definition would look like: |
those agents are constrained by the DICOM standard (same as PET) |
given the list: changes are
as a practical matter, I suggest to make a list (json to yml) with explicit reference to DICOM and validated checking values in there |
Hello,
We (@adhusch, @MeryemABBAD) are trying to convert several datasets from DICOM to the BIDS format. There are some datasets which use a contrast agent but it's not clear which one was used.
There is the ContrastBolusIngredient field in the meta-data but the documentation states that "Allowed values: IODINE, GADOLINIUM, CARBON DIOXIDE, BARIUM, XENON". Secondly there is the ceagend entity but the documentation states that it represents the ContrastBolusIngredient field. We assume that by using "ce: True" and "ContrastBolusIngredient: UNKNOWN" the dataset would not comply with BIDS anymore. How does BIDS handle these cases?
Thanks!
The text was updated successfully, but these errors were encountered: