Page MenuHomePhabricator

DICOM Query/Retrieve doesn't work at least in Ubuntu release installers
Closed, ResolvedPublic

Description

Someone wrote:

found out the problem with regards to MITK not doing query/retrieve to dicom server in Ubuntu. My dcmtk version did not have a dictionary, so you have to download https://raw.githubusercontent.com/InsightSoftwareConsortium/DCMTK/master/dcmdata/data/dicom.dic put it in a folder (e.g. /opt/dcmtk/dicom.dic) and then make sure to add export DCMDICTPATH=/opt/dcmtk/dicom.dic in your .bashrc

Event Timeline

kislinsk created this task.

The missing DCMTK dictionary in Linux and macOS installers seems to be the actual issue for some other tickets as well. We recently experienced this again with DICOM files from @s429r. Loading works on Windows in both build and installer versions, but only in build versions on Linux and macOS.

kislinsk raised the priority of this task from High to Unbreak Now!.Nov 23 2020, 11:57 AM
kislinsk edited projects, added MITK (v2021.02), Cleared; removed MITK.

Verified on macOS. I downloaded the dictionary and exported DCMDICTPATH. The MITK Workbench was able to load the DICOM files correctly afterwards. 🎉

Everything you need to know about the DICOM dictionaries and how DCMTK is handling them is described here: https://github.com/InsightSoftwareConsortium/DCMTK/blob/master/dcmdata/docs/datadict.txt

I quote:

By default, DCMTK uses an external data dictionary on Posix systems (Linux, Mac OS X, etc.) while a built-in dictionary is used on Windows systems. How these defaults can be changed or how both approaches can even be combined is further explained below.

I vote for the most straight forward option to also use built-in dictionaries on Linux and macOS. Related CMake variables are:

  • DCMTK_ENABLE_EXTERNAL_DICTIONARY
  • DCMTK_ENABLE_BUILTIN_DICTIONARY

Deleted branch bugfix/T26164-AlwaysUseDCMTKBuiltInDICOMDictionaries.

Let's see if the Nightly installers tomorrow morning are able to load Sarah's DICOM files. 🤞

tenor.gif (162×220 px, 247 KB)

Mixed feelings here: On one hand I'm happy that we, with a single trivial line of CMake code, finally fixed a decade old severe issue we had in MITK without really noticing it causing quite some inconvenience for our users and we were not able to reproduce it over the years. Maybe we even lost some users right before they actually used MITK because "it cannot open my DICOM files". On the other hand, we fixed a decade old severe issue with a single line of CMake code.

I guess most if not all of the linked tasks are resolved as well.

kislinsk claimed this task.

I would see it positiv. No sense crying over spilt milk. Thanks for solving that problem!