Project

General

Profile

Actions

Feature #818

open

Add support for multiple parameter values to SR Template TID 1500

Added by Jörg Riesmeier over 7 years ago. Updated about 7 years ago.

Status:
New
Priority:
Low
Category:
Library
Target version:
-
Start date:
2018-03-13
Due date:
% Done:

0%

Estimated time:
Module:
dcmsr, dcmsr/cmr
Operating System:
Compiler:

Description

CP-1705 introduces multiple values for the TID 1500 template parameter $Measurement (BCID 7468 "Texture​ Measurements" in addition to BCID 7469).
This is currently not supported by the implementation of the included templates (as C++ template classes):

class DCMTK_CMR_EXPORT TID1500_MeasurementReport
  : public DSRRootTemplate
{

  public:

    // type definitions
    typedef TID1411_VolumetricROIMeasurements<CID7469_GenericIntensityAndSizeMeasurements,
                                              CID7181_AbstractMultiDimensionalImageModelComponentUnits,
                                              CID6147_ResponseCriteria,
                                              CID7464_GeneralRegionOfInterestMeasurementModifiers>
            TID1411_Measurements;

    typedef TID1501_MeasurementGroup<CID7469_GenericIntensityAndSizeMeasurements,
                                     CID7181_AbstractMultiDimensionalImageModelComponentUnits,
                                     CID6147_ResponseCriteria,
                                     CID7464_GeneralRegionOfInterestMeasurementModifiers>
            TID1501_Measurements;

[...]
};
Actions #1

Updated by Jörg Riesmeier about 7 years ago

A possible solution would be to introduce a new method includeContextGroup() to the class DSRContextGroup, which allows for creating a hierarchical structure of nested context groups. The simplest form would be a (pseudo) context group that includes two other context groups, e.g. CID 7468 and 7469 (in order to solve the above described issue).

Actions #2

Updated by Jörg Riesmeier about 7 years ago

The "idea" from Note #1 will (most probably) not work, since a proper definition of the EnumType would be missing then in the "surrounding" context group (which includes the two others).

Actions

Also available in: Atom PDF