Add a dicom source. #1005
Add a dicom source. #1005
Conversation
|
As an implementation note, I had looked into using the dicomslide library rather than the wsidicom library. dicomslide requires numerous dicom fields to be set that are not and cannot be set via the wsi2dcm program, and therefore doesn't seem to be an easy solution. If we want to use dicomslide, we could augment the response from the dicomweb_client to fill in the fields dicomslide requires. This still will fail in the UIDs specified in the dicom files are not conformant to the DICOM spec, whereas wsidicom doesn't care. Further, the FrameOfReferenceUID that wsi2dcm generates isn't what dicomslide expects, so that would also have to be modified. The main virtue of dicomslide is that it can handle dicomweb and (I think) could write out data. Support for multiple Z slices or channels can be added with either library (I think), but requires samples to validate. |
This uses wsidicom to read dicom files. So far it has only been tested with files that were generated with the wsi2dcm docker from the 1.0.3 deb from https://github.com/GoogleCloudPlatform/wsi-to-dicom-converter, which seems to have issues with some files. This may be refactored to use a different base library, and internal details should be considered provisional. It needs to have internal metadata exposed.
manthey
force-pushed
the
dicom-source
branch
from
4bce481 to
5f0b007
Compare
Set limits on tile sizes.
This uses wsidicom to read dicom files. So far it has only been tested with files that were generated with the wsi2dcm docker from the 1.0.3 deb from https://github.com/GoogleCloudPlatform/wsi-to-dicom-converter, which seems to have issues with some files.
This may be refactored to use a different base library, and internal details should be considered provisional.