Page MenuHomePhabricator

[Dashboard] mitkDICOMLocaleTest_spacingOk_xx tests fail on nightly windows clients
Closed, ResolvedPublic

Description

see http://cdash.mitk.org/testSummary.php?project=1&name=mitkDICOMLocaleTest_spacingOk_CT&date=2012-06-05

File to load has been specified on commandline [PASSED]
Configuration:
MITK_USE_GDCMIO: true
GDCM_VERSION: 2.0.18

  • Changing locale from C to 'C'

Tue Jun 05 02:18:13 2012
1.66 core.io.dicomSeriesreader WARNING: Could not access tag (0018,0050): IDifyTagValue() illegaly called with empty tag value
1.66 core.io.dicomSeriesreader WARNING: Could not access tag (0018,0050): IDifyTagValue() illegaly called with empty tag value
1.66 core.io.dicomSeriesreader WARNING: Could not access tag (0018,0050): IDifyTagValue() illegaly called with empty tag value
1.66 core.io.dicomSeriesreader WARNING: Could not access tag (0018,0050): IDifyTagValue() illegaly called with empty tag val

Event Timeline

Who is the (new) DICOM-officer?

I developed most of that code and I'm still motivated to get most DICOM files loaded in a correct and meaningful way.

Today its only two machines:
http://cdash.mitk.org/testSummary.php?project=1&name=mitkDICOMLocaleTest_spacingOk_CT&date=2012-06-25

If somebody could provide me with information on what actually happens on the dartclient (segfault? where?) I'm happy to help. From just the test output, I don't have a clue though.

I´ll have a look on Wednesday...
and will keep you updated.

The behaviour occurs only on Windows-Debug builds. The reason is a Debug runtime check made by the VS ending in a pop-up message:

"Debug assertion failed!
[...]
invalid operator< "

The pop-up window awaits user interaction, hence without any, the test ends up in a timeout.

[cc3fa9]: Merge branch 'bug-12126-Failing-DICOMLocaleTest'

Merged commits:

2012-06-27 17:31:21 Jan Hering [fef69c]
Making GdcmSortFunction a proper operator< type

  • replacing 'return true' which was independent of the parameter order by
  • a comparison of the mandatory and unique tag values SOPInstanceUID

Test no more failing on nightly clients. Issue solved. Closing bug.