Page MenuHomePhabricator
Create Task
Maniphest T10488

Pixel value on the status bar with segmentation
Closed, ResolvedPublic
Actions

Description

If an image and a segmentation is made visible then the pixel value displayed on the status bar is from the segmentation, not from the base image. This is independent of which image is selected.

This is a little bothering, since if you want to see a pixel value in a ROI than you have hide the ROI first. If the ROI is checked on, you see 1 or 0 on the status bar.

I note that the statusbar is updated from two places in MITK, and both are active. *QmitkStdMultiWidget and mitkSliceNavigatorController if I remember well.) So if this will be fixed, it has to be fixed in both places, or to remove one of them.

Related Objects

Event Timeline

espak added projects: MITK, Bugzilla.Dec 17 2011, 1:12 AM
• fetzer added a subscriber: • fetzer.Nov 27 2012, 6:39 PM

Changed Target Milestone to upcoming release

• fetzer added a comment.Dec 18 2012, 3:58 PM

Updating target milestone to upcoming release

• iszatt added a subscriber: • iszatt.Jan 23 2013, 5:42 PM

New remote branch pushed: bug-10488-PixelValueOnStatusBarWithSegmentation

• bauerm added a subscriber: • bauerm.Mar 25 2013, 6:51 PM

Fixed in HandleCrosshairPositionEventDelayed() in QmitkStdMultiWidget wich is called after mitkSliceNavigatorController. Now if you click on a binary image it searches for source images of the binary image. If there is a source image the image information of the source image will be displayed in the status bar otherwise the image information of the binary image will be displayed.

I would suggest to open another bug for mitkSliceNavigationController and disable the execution of the event if QmitkStdMultiWidget is in use and HandleCrosshairPositionEventDelayed consumes the crosshair event already.

• bauerm added a comment.Mar 25 2013, 6:59 PM

New remote branch pushed: bug-10488-PixelValueOnStatusBarWithSegmentationRebased

• fetzer added a comment.Mar 26 2013, 2:50 PM

Yep I totally agree with you. The duplicate code in the QmitkStdMultiWidget and the SliceNavigationController is not nice, especially the duplicate execution of this code. Because of the latter: Could you please add the binary image stuff to the SliceNavigationController as well? Currently since the SliceNavigationController is setting the status bar text either the displayed pixel value is constantly switching between the binary value and the image's value.

• bauerm added a comment.Apr 15 2013, 1:14 PM

Ok I added the binary image stuff to SliceNavigationController as well and pushed it onto remote branch.

nolden added a subscriber: nolden.Apr 15 2013, 2:07 PM

Could you please add your manual testing description to the application checklist? You could ask Andreas for help if necessary.

• bauerm added a comment.Apr 15 2013, 3:50 PM

The testing description is added to the list. In my oppinion it's sufficiant to test the whole szenario.

• bauerm added a comment.May 6 2013, 10:37 AM

[443b01]: Merge branch 'bug-10488-PixelValueOnStatusBarWithSegmentationRebased'

Merged commits:

2013-04-15 12:59:11 Michael Bauer [9b56fa]
Implemented new method GetTopLayerNode to calculate the highest layer image in the
DataStorage which isn't a helper object and returns it. This method is used in ExecuteAction.

Adjusted the way to display status text in ExecuteAction.
Now the behavior of displaying is the same as in QmitkStdMultiWidget.

2013-03-25 16:20:37 Michael Bauer [b7bd1f]
Adjusted logic in HandleCrosshairPositionEventDelayed().
Now if you click on a binary image it searches for source images of the binary image.
If there is a source image the image information of the source image will be displayed in the status bar otherwise
the image information of the binary image will be displayed.

2013-02-13 18:03:00 Michael Bauer [5bb24a]
Added new method for layer computation.
Added some code to prevent the StatusBar from displaying image information
of a binary image if the binary source image is available.
In case that the source image is available the status bar will display the
pixel values of the source image.
You will see the binary image on top of the source image but the pixel values
of the source image in the StatusBar.

2013-01-23 17:41:56 Justin Iszatt [98e61d]
Started working on the bug. look at the qmitkstdmultiwidget at line 1626

kislinsk edited projects, added MITK (2013-06); removed MITK.Aug 1 2016, 10:47 PM
kislinsk mentioned this in T27033: Clean up stale remote branches.Jan 25 2020, 9:01 PM
MITK · MIC · IMSY · DKFZ