Page MenuHomePhabricator

LevelWindow - "Use whole image grayvalues" does not work with multi-component images properly
Closed, ResolvedPublic

Description

For 3D and 3D+t (the currently displayed timestep), the 'Use whole image gray values' does what it states. However, this feature does not work for a multi component image (like the Diffusion image)

Here, regardless the selected component, the values are taken from the first one
(or all values, I cannot tell it as all diffusion images have b=0 images as first component which is then the one with the biggest intensity range)

Event Timeline

I will take this bug. Is the wanted behavior that the level window shall display the _current_ component's gray values?

User wirkert has pushed new remote branch:

bug-18777-level_window_multi_component_images

User wirkert has pushed new remote branch:

bug-18777-v2_level_window_multi_component_images

Actual behavior:
Level window does not adapt for multi component images.

Expected behavior:
Should adapt.

Cause of the bug:
level window doesn't have respective functionality

Proposed solution:
see function mitk::LevelWindowManager::RecaluclateLevelWindowForSelectedComponent
and changes to SetAuto (component is now passed to SetAuto and Statistics calculators)

Affected classes:
git diff

How will the bugfix get tested:
CppUnit test has been implemented

[7e3253]: Merge branch 'bug-18777-v2_level_window_multi_component_images'

Merged commits:

2015-08-19 16:40:53 Sebastian Wirkert [20e42d]
Moved new level window test to cpp unit format.


2015-08-19 16:05:00 Sebastian Wirkert [852968]
Wrote test for changes in level window manager.


2015-08-05 15:04:09 Sebastian Wirkert [c5a9a2]
Implemented the function that the levelwindow adapts to the selected component.
Works.


2015-06-24 15:22:24 Sebastian Wirkert [f42885]
Intermediate checkin: added listerner to the component changed event.
Now we have to do the correct recalcuations in case that event happens


2015-06-24 14:55:50 Sebastian Wirkert [ce37d0]
small refactoring to get started.