gpilab / core-nodes

The core nodes are a collection of basic data manipulation and visualization algorithms.
http://docs.gpilab.com/en/develop/NodeDev/devguide.html
BSD 3-Clause "New" or "Revised" License
12 stars 6 forks source link

DICOM Reader #20

Open borupdaniel opened 4 years ago

borupdaniel commented 4 years ago

A node that reads DICOM images and outputs them for use in the canvas. Could also include an option to save in another format like .npy.

rkrobison commented 4 years ago

@borupdaniel, a DICOM reader for GPI exists as part of the GPIdicom library.

borupdaniel commented 4 years ago

Thanks Ryan, I'll have a look at that. I think this would be of general interest — would you be okay with folding it into gpi_core?

rkrobison commented 4 years ago

Sure, no problem. Although, I think the writer still needs some bugs worked out, and there are some places where efficiency could be a lot better.

On Mar 25, 2020, at 1:30 PM, Daniel Borup notifications@github.com wrote:

Thanks Ryan, I'll have a look at that. I think this would be of general interest — would you be okay with folding it into gpi_core?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/gpilab/core-nodes/issues/20#issuecomment-604070490, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACPUR6FNXTNHA3D47F7IJRLRJJSWRANCNFSM4IU76UYA.

borupdaniel commented 4 years ago

Hey @rkrobison — have you used the ReadDICOM node from GPIdicom for Enhanced DICOM images before? I'm finding it works ok for a single slice, but not in the case of multi-slice enhanced DICOM.

rkrobison commented 4 years ago

Hi @borupdaniel, that is a good question. I think I might not have tested it with Enhanced DICOM to date.

I think it is supported by pydicom, but it might require some different handling of the data in the GPI node. I don't have any enhanced DICOM data, but I could try to take a look sometime if you want to share some.