radmedres / clmedview

An attempt to create a cross-platform medical image viewer specialized in drawing "regions of interests".
GNU General Public License v3.0
3 stars 2 forks source link

Window Width/level not working.... #5

Closed minovations closed 9 years ago

minovations commented 9 years ago

Hi Folks,

I started clmedview and loaded an image into memory. By clicking the right button, I wanted to change the window width and level settings. This is not possible!!!

By reading the source code, I understood how window width and level would working. The ww-wl request is not activated, because the "resources->ps_ActiveLayer" is NULL.

Could the "resources->ps_ActiveLayer" be set to the original image when loading the image?

Grts Jos

roelj commented 9 years ago

Yes, absolutely.

I think it would be great if the original layer could be selected in the sidebar by default as well. For this I'd like to propose this patch: https://github.com/roelj/clmedview/commit/db40b89ed32c69f512e1dc878bfdcc7f2f74a04a

I can send send a pull request when this is what you'd like to see as well.

minovations commented 9 years ago

This seems to be what the viewer should do, do you create a pull request?