question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

SEG source series should also be parsed from SourceImageSequence

See original GitHub issue

Request

SourceImageSequence should be parsed to determine the image series that is associated with segmentation.

Why should we prioritize this feature?

As observed from debugging https://github.com/ImagingDataCommons/IDC-WebApp/issues/238, segmentations from TCIA ISPY collection are using SourceImageSequence, not ReferencedImageSequence to refer to the image series segmented.

In the retrospect, I wonder if the approach used in I-SPY is the correct one. We use ReferencedImageSequence in dcmqi, and @dclunie never called out for using SourceImageSequence, so maybe what we do in dcmqi is still the right way to do it. Need to check with David.

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:7 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
fedorovcommented, Aug 28, 2020

@JamesAPetts yes, when I read it, I had the same thought, but it is the case that what we have in dcmqi was checked and double checked by David Clunie via our joint work on many datasets we shared created by dcmqi, so I am puzzled … I will ask David when I get a chance.

0reactions
JamesAPettscommented, Sep 2, 2020

Going to close this for now, I’ve made the required upgrades to read all the data I can see on IDC. I tested 20-30 studies looking for edge cases but all seem good now. If you find an edge case/if there is any fundemental issue you encounter please open another ticket.

Thanks.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Source Image Sequence Attribute - DICOM Standard Browser
A Sequence that identifies the Images that were used to derive this Image. One or more Items shall be included in this Sequence....
Read more >
Unable to load DICOM segmentation file - #12 by pieper
The segmentation file has several errors. Several Type 1 (required) DICOM fields are missing, but the reader is quite robust so it could...
Read more >
highdicom Documentation
Creating and parsing DICOM objects using the highdicom package. 3.1 Creating Segmentation (SEG) images. Derive a Segmentation image from a ...
Read more >
VVDicom conformance statement_Changes - GE Healthcare
The reader of this DICOM Conformance Statement should be aware that ... 4.5.6.1.1.3 Derivation Description and Source Image Sequence.
Read more >
DICOM Conformance Statement
The Conformance Statement should be read and understood in ... The Storage SCP AE can also respond to C-ECHO requests as a Verification...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found