Skip to content
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

Closed
RaphaelMaser opened this issue Jul 8, 2023 · 8 comments · Fixed by #1547
Closed

Unknown Contrast Agent in BIDS #1542

RaphaelMaser opened this issue Jul 8, 2023 · 8 comments · Fixed by #1547

Comments

@RaphaelMaser
Copy link

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!

@Remi-Gau
Copy link
Collaborator

Remi-Gau commented Jul 8, 2023

How does BIDS handle these cases?

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

ds001/sub-01
├── anat
│   ├── sub-01_ce-foo_T1w.json
│   ├── sub-01_ce-foo_T1w.nii.gz
│   └── sub-01_inplaneT2.nii.gz
$ cat ds001/sub-01/anat/sub-01_ce-foo_T1w.json 

{"ContrastBolusIngredient": "foo"}

@RaphaelMaser
Copy link
Author

Thanks for you answer!
So, if I understand you correctly then this would not be a future-proof way of indicating the contrast agent. Is there a better way to include the information about the contrast agent without (potentially) loosing bids validity in the future?

@Remi-Gau
Copy link
Collaborator

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?

@effigies
Copy link
Collaborator

Seems like there's a legitimate use case, so we should support it. Also seems like there should be a none value, to distinguish scans with/without in studies that have both.

@RaphaelMaser
Copy link
Author

Thanks for your help, including these options in the specification would be great!

@Remi-Gau
Copy link
Collaborator

This is what the new metadata definition would look like:

https://bids-specification--1547.org.readthedocs.build/en/1547/glossary.html#contrastbolusingredient-metadata

@CPernet
Copy link
Collaborator

CPernet commented Jul 29, 2023

those agents are constrained by the DICOM standard (same as PET)
list here https://dicom.nema.org/medical/dicom/current/output/chtml/part16/sect_CID_12.html#sect_CID_12

@CPernet
Copy link
Collaborator

CPernet commented Jul 29, 2023

given the list: changes are

  • barium --> Barium Sulfate
  • iodine -- well there are many variations just iodine does not look correct
  • xenon does not look supported ? where is this coming from
  • none -- fine
  • UNKNOWN -> contrast agent (there is such a general term in the list)

as a practical matter, I suggest to make a list (json to yml) with explicit reference to DICOM and validated checking values in there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants