displaying images

[ultradian]

I'm playing with DICOM.jl a bit and was writing some code to display raw images. Is this something useful to merge in with the package, and if so, what suggestions do people have about useful display formats or how this package should interface with other packages?

[ihnorton]

I'm playing with DICOM.jl a bit and was writing some code to display raw images. Is this something useful to merge in with the package

I think that's out of scope (take a look at some of the other JuliaIO packages -- they also stick to I and O).

how this package should interface with other packages?

If you are interested in adding Images.jl support, that would be really neat! Then you'd get visualization "for free" from several different Julia viz tools. Making Images.jl a dependency in order to facilitate that would be fine.

…

On Wed, Mar 29, 2017 at 6:14 PM, ultradian ***@***.***> wrote: I'm playing with DICOM.jl a bit and was writing some code to display raw images. Is this something useful to merge in with the package, and if so, what suggestions do people have about useful display formats or how this package should interface with other packages? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#15>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAUAGss86ZhWxi4zwSEAOVK34K-FKeJgks5rqtfBgaJpZM4Mtn8L> .

[ultradian]

@ihnorton, I'm confused. When you talk about Images support, do you mean creating a display or show function in DICOM that uses Images.jl? That is the same thing I meant, when I was talking about raw image display. I'm not sure if that belongs here or in JuliaImages as you mentioned it was out of scope.

[SimonDanisch]

I think @ihnorton meant, that if you create the integration with Images.jl, so that e.g. load("test.dicom") returns an image (which is nowadays just an array of colors), all plotting and visualization packages can already display those and defining show yourself isn't needed!

[ultradian]

Thanks for the clarification, @SimonDanisch. I have created a new thread #16 with a proposal to split up the data to make this load/image display easier before creating a PR. What is the process to get push access here in JuliaIO? As a newbie, I appreciated your explanations of how thing work in #14. I don't really know who has push access for this repo besides @ihnorton which I assume you set up as the JuliaIO member who was most involved.