DICOM @ OFFIS

I can add the +on2 option and get a 16 bit jpeg image No, +on2 creates a PNG image with 16 bits per sample. Here's the relevant excerpt from the dcmj2pnm manpage: image format: +op --write-raw-pnm write 8-bit binary PGM/PPM (default for files) +opb --write-8-bit-pnm write 8-bit ASCII PGM/PPM (defau...

One question, what do you mean by "is already the thread-safe variant of a global variable"? It's definitely not thread-local, at least not in version 3.6.5, as there was a thread influencing another. Thread safety does not always mean thread-local storage. In this case, it's a global variable wher...

I can't help wondering why this is a global setting in the first place, though, rather than something like a flag in DcmDocument or similar that can be independently set for different contexts. You mean DcmDataset? This is mainly for historical reasons. When the DICOM dataset parser in "dcmdata" wa...

Changing global settings (e.g. OFGlobal instances) within a class is never a good idea, also in this case. Therefore, I would agree that removing the line(s) from DcmSCP would be the preferred solution. DcmStorageSCP has a "bit preserving mode" (similar to the one of storescp), so you should documen...

I've also found that DcmSCP does the following in DcmSCP::openListenPort(): // make sure not to let dcmdata remove trailing blank padding or perform other // manipulations. We want to see the real data. dcmEnableAutomaticInputDataCorrection.set(OFFalse); I'll ask the original author of this class wh...

My first idea was: Could you print the value of SOPClassUID stored in "dataset" or the entire "dataset" before and after calling "sr->read()"? But then, the following came to my mind: Could you check what the value of dcmEnableAutomaticInputDataCorrection.get() is? Maybe, your problem is related to ...

According to a statement of Mr. Riesmeier the timeout for the network is set to 60s. I also get a corresponding message in my log files. 2023-03-03 10:22:37.553 DEBUG: setting network send timeout to 60 seconds 2023-03-03 10:22:37.553 DEBUG: setting network receive timeout to 60 seconds Which of th...

From the log, it looks like the SCU is aborting the Association after the SCP (wlmscpfs) accepted the Association. The proposed Verification SOP Class is accepted with Explicit VR Little Endian Transfer Systax. Apart from the A-ABORT message that is sent from the SCU, I only find the following a bit...

Here is the corresponding commit: http://git.dcmtk.org/?p=dcmtk.git;a=com ... bb8266cd45

The new helper function should be available in DCMTK's public git repository in a few days. A new DCMTK release is expected in spring/summer this year.

Thank you for your report. I don't think that this was on purpose. I'll add this helper function to the DcmItem class.

Could you please provide the log output with debug messages enabled, i.e. when starting wlmscpfs with option --debug (or -d).

Did you create a "lockfile" in the data directory? See documentation for details.

See also DCMTK Feature #1041: https://support.dcmtk.org/redmine/issues/1041

Thank you for the feedback. I've just committed the required changes to the internal "testing" branch of the DCMTK. The changes should be visible in the public git repository soon.

Thank you for your report and detailed analysis. It seems that no one ever used this method before - including the author of this class :? The version I would propose is the following: OFCondition DRTImage::read(DcmItem *dataset) { /* get rid of the old image before we mess with its dataset */ reset...