View Issue Details

IDProjectCategoryView StatusLast Update
0003255Slicer4Module DICOMpublic2018-05-29 23:18
ReporterfedorovAssigned Tomillerjv 
PrioritynormalSeverityminorReproducibilityalways
Status acknowledgedResolutionopen 
Product Version 
Target VersionbacklogFixed in Version 
Summary0003255: Create DICOM Seris output has multiple standard non-compliance issues
Description

As reported by David Clunie's dciodvfy:

Error - Value invalid for this VR - (0x0008,0x0030) TM Study Time TM - Trailing character invalid for this VR = '' (0)
Error - Value invalid for this VR - (0x0008,0x0050) SH Accession Number SH - Trailing character invalid for this VR = '' (0)
Warning - Value dubious for this VR - (0x0008,0x0090) PN Referring Physician's Name PN [0] = <Unknown> - Retired Person Name form
Error - Value invalid for this VR - (0x0008,0x0090) PN Referring Physician's Name PN - Trailing character invalid for this VR = '' (0)
Warning - Value dubious for this VR - (0x0010,0x0010) PN Patient's Name PN [0] = <Anonymous-MR> - Retired Person Name form
Error - Value invalid for this VR - (0x0010,0x0032) TM Patient's Birth Time TM - Trailing character invalid for this VR = '' (0)
Error - Value invalid for this VR - (0x0010,0x0040) CS Patient's Sex CS - Trailing character invalid for this VR = '' (0)
Error - Value invalid for this VR - (0x0018,0x0050) DS Slice Thickness DS - Trailing character invalid for this VR = '' (0)
Error - Value invalid for this VR - (0x0018,0x5100) CS Patient Position CS - Trailing character invalid for this VR = '' (0)
Error - Value invalid for this VR - (0x0020,0x0013) IS Instance Number IS - Trailing character invalid for this VR = '' (0)
Error - Dicom dataset contains invalid data values for Value Representations
Warning - Retired attribute - (0x0032,0x4000) LT Study Comments
Warning - Dicom dataset contains retired attributes
CTImage
Error - Missing attribute Type 2C Conditional Element=<Laterality> Module=<GeneralSeries>
Error - Missing attribute Type 2 Required Element=<KVP> Module=<CTImage>
Error - Missing attribute Type 2 Required Element=<AcquisitionNumber> Module=<CTImage>
Warning - Attribute is not present in standard DICOM IOD - (0x0032,0x4000) LT Study Comments
Warning - Dicom dataset contains attributes not present in standard DICOM IOD - this is a Standard Extended SOP Class

Here's the dcmdump output (some errors are obvious there):

Dicom-File-Format

Dicom-Meta-Information-Header

Used TransferSyntax: Little Endian Explicit

(0002,0000) UL 258 # 4, 1 FileMetaInformationGroupLength
(0002,0001) OB 00\01 # 2, 1 FileMetaInformationVersion
(0002,0002) UI =CTImageStorage # 26, 1 MediaStorageSOPClassUID
(0002,0003) UI [1.2.826.0.1.3680043.2.1125.1.63005834202781640426469379502873233] # 64, 1 MediaStorageSOPInstanceUID
(0002,0010) UI =LittleEndianImplicit # 18, 1 TransferSyntaxUID
(0002,0012) UI [1.2.826.0.1.3680043.2.1143.107.104.103.115.2.0.17.111.124.113] # 62, 1 ImplementationClassUID
(0002,0013) SH [GDCM 2.0.17] # 12, 1 ImplementationVersionName
(0002,0016) AE [GDCM/ITK 4.4.0] # 14, 1 SourceApplicationEntityTitle

Dicom-Data-Set

Used TransferSyntax: Little Endian Implicit

(0008,0008) CS [ORIGINAL\PRIMARY\AXIAL] # 22, 3 ImageType
(0008,0016) UI =CTImageStorage # 26, 1 SOPClassUID
(0008,0018) UI [1.2.826.0.1.3680043.2.1125.1.63005834202781640426469379502873233] # 64, 1 SOPInstanceUID
(0008,0020) DA [20060101] # 8, 1 StudyDate
(0008,0030) TM [010100.000000^@] # 14, 1 StudyTime
(0008,0050) SH [1^@] # 2, 1 AccessionNumber
(0008,0060) CS [CT] # 2, 1 Modality
(0008,0070) LO [GE Medical Systems] # 18, 1 Manufacturer
(0008,0090) PN [Unknown^@] # 8, 1 ReferringPhysicianName
(0008,1030) LO [None] # 4, 1 StudyDescription
(0008,103e) LO [None] # 4, 1 SeriesDescription
(0008,1090) LO [None] # 4, 1 ManufacturerModelName
(0010,0010) PN [Anonymous-MR] # 12, 1 PatientName
(0010,0020) LO [123456] # 6, 1 PatientID
(0010,0030) DA [20060101] # 8, 1 PatientBirthDate
(0010,0032) TM [010100.000000^@] # 14, 1 PatientBirthTime
(0010,0040) CS [M^@] # 2, 1 PatientSex
(0010,4000) LT [None] # 4, 1 PatientComments
(0018,0050) DS [3.00004^@] # 8, 1 SliceThickness
(0018,5100) CS [HFS^@] # 4, 1 PatientPosition
(0020,000d) UI [1.2.826.0.1.3680043.2.1125.1.44544713079695912631873658822636477] # 64, 1 StudyInstanceUID
(0020,000e) UI [1.2.826.0.1.3680043.2.1125.1.50013295386092768399741099164944599] # 64, 1 SeriesInstanceUID
(0020,0010) SH [123456] # 6, 1 StudyID
(0020,0011) IS [123456] # 6, 1 SeriesNumber
(0020,0013) IS [1^@] # 2, 1 InstanceNumber
(0020,0032) DS [-71.5342-75.4786-61.8724] # 26, 3 ImagePositionPatient
(0020,0037) DS [1\0\0\0\1\0] # 12, 6 ImageOrientationPatient
(0020,0052) UI [1.2.826.0.1.3680043.2.1125.1.57331868242101617140230241962633483] # 64, 1 FrameOfReferenceUID
(0020,1040) LO [SN] # 2, 1 PositionReferenceIndicator
(0028,0002) US 1 # 2, 1 SamplesPerPixel
(0028,0004) CS [MONOCHROME2] # 12, 1 PhotometricInterpretation
(0028,0010) US 512 # 2, 1 Rows
(0028,0011) US 512 # 2, 1 Columns
(0028,0030) DS [0.3125\0.3125] # 14, 2 PixelSpacing
(0028,0100) US 16 # 2, 1 BitsAllocated
(0028,0101) US 16 # 2, 1 BitsStored
(0028,0102) US 15 # 2, 1 HighBit
(0028,0103) US 1 # 2, 1 PixelRepresentation
(0028,1052) DS [0] # 2, 1 RescaleIntercept
(0028,1053) DS [1] # 2, 1 RescaleSlope
(0032,4000) LT [None] # 4, 1 RETIRED_StudyComments
(7fe0,0010) OW 0000\0000\0000\0000\0000\0000\0000\0000\0000\0000\0000\0000\0000... # 524288, 1 PixelData

TagsNo tags attached.

Relationships

related to 0004344 assignedjcfr Create a DICOM Series module makes invalid DICOM 

Activities

pieper

pieper

2013-07-29 11:23

administrator   ~0009252

Hi Jim -

You inherited the Create A DICOM Series from Ol' Bill so I assigned this to you. Have there been any fixes at the ITK/GDCM level that might address this?

Longer term as part of the U24 (assuming it's funded) we'll need to make another pass at all the DICOM export functionality in order to smarten things up.

For example, David's dciodvfy code probably wasn't able to detect that the image data is most likely an MR that was exported with a Modality of CT ;)

-Steve

fedorov

fedorov

2013-07-29 12:01

developer   ~0009254

Steve, Jim - part of it is actually a lot simpler than wrong modality: something in the ITK guts adds extra trailing character after the tag value (for example, look at AccessionNumber tag above).

pieper

pieper

2013-07-29 12:47

administrator   ~0009259

Hi Andrey - I was actually arguing that it's actually a lot more complicated than simply the extra character or the modality. But fixing the trailing character is a simple fix.

Do we know that this causes any issue in practice? Do PACS refuse to accept the images with the trailing spaces?

-Steve

fedorov

fedorov

2013-07-29 12:57

developer   ~0009260

Steve: I agree there's a lot more.

I sent the resulting DICOMs to Robert Cormack, he was going to let me know if their system accepts those.

millerjv

millerjv

2013-07-29 14:13

developer   ~0009263

Can't speak to current DICOM libraries, but at one point in time CreateADICOMSeries was "designed" to have the smallest set of DICOM tags needed to import into several PACS and the AW.

Note that DICOM validator tools are great to identify problems wrt the standard. But over the years our biggest problem was figuring out what needed to be in the DICOM to have it accepted by the PACS and AW. For instance, some systems require patient birthdate to be a valid date or it won't import the series.

I'll take a look at the validator output.

fedorov

fedorov

2013-07-31 06:33

developer   ~0009297

Jim: here the validator complains (in particular) about the characters that cannot be present for certain VR types, look at the StudyTime content as an example - the value is invalid. This particular issue appears to be very basic.

lassoan

lassoan

2018-05-29 23:18

developer   ~0015779

Retargeting all old issues to backlog.

Issue History

Date Modified Username Field Change
2013-07-26 11:47 fedorov New Issue
2013-07-26 11:47 fedorov Status new => assigned
2013-07-26 11:47 fedorov Assigned To => pieper
2013-07-29 11:20 pieper Assigned To pieper => millerjv
2013-07-29 11:23 pieper Note Added: 0009252
2013-07-29 11:23 pieper Status assigned => acknowledged
2013-07-29 12:01 fedorov Note Added: 0009254
2013-07-29 12:47 pieper Note Added: 0009259
2013-07-29 12:57 fedorov Note Added: 0009260
2013-07-29 14:13 millerjv Note Added: 0009263
2013-07-31 06:33 fedorov Note Added: 0009297
2017-02-16 11:24 fedorov Relationship added related to 0004344
2018-05-29 23:18 lassoan Target Version => backlog
2018-05-29 23:18 lassoan Note Added: 0015779