Markus D. Herrmann

Results 258 comments of Markus D. Herrmann
trafficstars

@malaterre See [this comment](https://github.com/SimpleITK/SimpleITK/issues/440#issuecomment-383178644) on the corresponding SimpleITK github issue on how to generate the enhanced image object.

Reading multi-frame images now works with the 5.0 Beta 1 release. However, the values of the spacing, origin and direction attributes are not set correctly but represent the defaults.

@thewtex thanks for your follow up. I just tested with version `5.0b3`. The enhanced image is now read correctly via the `imread()` function: ```python import itk enhanced_image = itk.imread(enhanced_filename) enhanced_image.GetSpacing()...

@thewtex please see [this comment](https://github.com/SimpleITK/SimpleITK/issues/440#issuecomment-383178644) on issue #440 of SimpleITK repository on how to reproduce

I tested with series `1.3.6.1.4.1.9328.50.50.104552199813796461463367647403696421653` of study `1.3.6.1.4.1.9328.50.50.78410421624738603252174560245217471974`. Expected spacing: `itkVectorD3 ([0.859375, 0.859375, 7.5])`

Based on my understanding, the retrieve study, series, and instance transactions of the DICOM web studies service shall serve `"multiplart/related; type="application/dicom"`. At least this is how it is intended. Unfortunately,...

The retrieve instance resource is a bit of a mess. In principle, clients can request instances as Part10 files (`multipart/related; type="application/dicom"; transfer-syntax=*`) or as bulkdata (`multipart/related; type="application/octet-stream"; transfer-syntax=*` for uncompressed...

I am sorry @dimrozakis, we completely forgot about that when implementing the file client. Splitting the two clients into separate packages would unfortunately also not be ideal at this point,...

Thanks @sedghi for reporting this interesting observation and for the technical deep dive. [DICOM PS3.18 Section 8.7.1 Multipart Media Types](https://dicom.nema.org/medical/dicom/current/output/chtml/part18/sect_8.7.html#sect_8.7.1) states > The "type" parameter is required. It contains the...

Thanks everyone for providing additional background and feedback. Given all the great input, I suggest adding a parameter to the constructor of the `DICOMwebClient` class to control the behavior (e.g.,...