-
Notifications
You must be signed in to change notification settings - Fork 61
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
Establish recommendations/features when SEG and image geometries are very different #489
Comments
Following on the comment from @dclunie in the aforementioned page ...
David, this looks exactly like what we need! When you say "perhaps consider", are you suggesting we can just start using that sequence, or that we would need to have a CP to add it to SEG? |
It was added only for RT Structure Sets, since RT planning was the use case. To use it in SEG would require a new CP, and the addition would need to be reworded to explain what it "means" in a SEG usage context. |
@dclunie here is the rationale for this addition. Do you need anything to proceed with the CP? Rationale: Explicit reference of the series that was used to generate SEG is important for a variety of reasons. A typical DICOM studies contains multiple series, but only a subset of those may be used for performing segmentation. These could be series that have different contrast or acquisition protocol. There could also be motion or deformation of the anatomy being segmented across different series. It is important to communicate what series were used for segmentation for the purposes of provenance and reproducibility. Limitations of the current SEG: Currently, it is impossible to reference the series that was used to create SEG directly. Instead, it is possible to reference individual instances that were used for creating a specific SEG frame in the per-frame functional groups sequence. This approach is problematic for several reasons. First, it requires establishing frame-level references, which may be difficult or impossible (e.g., in the situation where orientation of the SEG frames is orthogonal to that of the image slices). Second, this may lead to prohibitive size of the We propose to allow for referencing the series that was used for creating SEG at the series level using SourceSeriesInformationSequence and SourceSeriesSequence (CP 2296 Provide additional ROI parameters to avoid parsing strings). |
@dclunie says we can include references to the segmented series in the shared FG sequence. |
More specifically, the DerivationImageMacro (that includes the references to all the images in the series) can be included at the Shared FG level, rather than including a reference to one image (or all the images) in the Per-Frame FG for each segmented frame - note that the Series per se is still not referenced by this means, since the references are all at the instance level, but the required (satisfied conditional) presence of the Common Instance Reference Module will include the referenced Series information, if that is what is wanted by the consumer. |
@pieper @ReubenDo I finally added the feature that allows to include references to the source images in the cases where segmentation slice orientation does not match that of the source image. All you need to do is add the flag I tested this on one of the segmentations from the ReMIND collection, and it works as expected (including that MR series is loaded automatically in Slicer when you load SEG).
Result:
You will need to use the latest release https://github.com/QIICR/dcmqi/releases/tag/latest or v1.3.4. I don't know if you are still working on that project or have plans to update the dataset, but in case you do - now you can. |
Triggered by this use case: https://projectweek.na-mic.org/PW40_2024_GranCanaria/Projects/LinkingSegmentationAndImagingDataWithDifferentGeometriesUsingDcmqi/
Currently, references are established at the individual segment frame level, and referenced instances are then included in the
ReferencedSeriesSequence > ReferencedInstanceSequence
. This approach will not be practical when the source has a different geometry, since establishing correspondence between segmentation frames and source images will be either not practical (we we ask users to define such correspondence while parameterizing conversion) or under-defined and complex (would it be fair to assume that any slice intersecting the SEG frame should be referenced? if we have any significant number of frames, the size of these references will be prohibitive).Ideally, it would be nice to identify a mechanism where we could at the series level reference the segmented image series in SEG. @dclunie is there any way to do this?
The text was updated successfully, but these errors were encountered: