feat: Enhance orientation and position extraction from DICOM datasets… #2241
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
… using Per-frame Functional Groups Sequence
Context
This PR addresses the issue of missing ImageOrientationPatient, ImagePositionPatient, and PixelSpacing in enhanced multiframe DICOM datasets (e.g., CT/MR) where these values are not present at the top level of the dataset.
When loading volumes from multiframe images, cornerstone failed to extract required spatial metadata, leading to errors such as:
ImageOrientationPatient is undefined
This happened because the current implementation only supports:
Enhanced DICOM datasets often store these values inside the Per-frame Functional Groups Sequence (5200,9230) or Shared Functional Groups Sequence (5200,9229), which were not considered until now.
Changes & Results
Added fallbacks as last resort in:
These fallbacks are only triggered when no value is found in standard locations, preserving existing logic and behavior for non-enhanced DICOMs.
Checklist
PR
semantic-release format and guidelines.
Code
etc.)
Public Documentation Updates
additions or removals.
Tested Environment