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

include coil info parameter #51

Open
raamana opened this issue Sep 7, 2023 · 3 comments
Open

include coil info parameter #51

raamana opened this issue Sep 7, 2023 · 3 comments

Comments

@raamana
Copy link
Contributor

raamana commented Sep 7, 2023

we need to add coil info parameter(s) for protocol compliance checks.

Getting info on this is not easy and varies across vendors unfortunately (see here), but for siemens, it seems to be at 0x0051, 0x100F, so it should be easy for us to add.

@raamana
Copy link
Contributor Author

raamana commented Sep 7, 2023

some example codes/variations for coil string:

32 ch: HEA; HEP
64 ch: HC1-7; HC1-7; NC1,2
20 ch: HE1-4

relevant BIDS meta data tag: ReceiveCoilActiveElements

@sinhaharsh
Copy link
Collaborator

Added DICOM tag in config file

@raamana
Copy link
Contributor Author

raamana commented Nov 7, 2023

looks like we need to properly decode the coil string in dicom header, and implement the equality condition in a more detailed manner (beyond simple string match)

it would be ideal to make it work for all body organs, not just brain:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6175221/table/jmri26187-tbl-0001/?report=objectonly

@raamana raamana reopened this Nov 7, 2023
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

No branches or pull requests

2 participants