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

enhanced dicoms (XA) and sequence_name in PulseSequenceName dicom tag #752

Open
1 of 2 tasks
bpinsard opened this issue Apr 16, 2024 · 0 comments · May be fixed by #753
Open
1 of 2 tasks

enhanced dicoms (XA) and sequence_name in PulseSequenceName dicom tag #752

bpinsard opened this issue Apr 16, 2024 · 0 comments · May be fixed by #753

Comments

@bpinsard
Copy link
Contributor

bpinsard commented Apr 16, 2024

Summary

Finally got some Siemens XA dicoms in hands, and the sequence_name is now in (0018,9005) PulseSequenceName dicom tag, resulting in the seqinfo sequence_name to be empty.
I can submit a patch to the following, to extract this value when the other regular tags are missing.

if dcminfo.get([0x18, 0x24]):
# GE and Philips
sequence_name = dcminfo[0x18, 0x24].value
elif dcminfo.get([0x19, 0x109C]):
# Siemens
sequence_name = dcminfo[0x19, 0x109C].value
else:
sequence_name = ""

Platform details:

Choose one:

  • Local environment
  • Container

ubuntu, python3.10

  • Heudiconv version:

1.1.0

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.

1 participant