Page MenuHomePhabricator

Segmentation: Region Growing tool almost unusable
Closed, ResolvedPublic

Assigned To
None
Authored By
baumhaue
Dec 21 2009, 9:30 PM
Referenced Files
F420: patch-3202-tool.patch
Jan 6 2010, 6:20 PM
F419: patch-3202-core.patch
Jan 6 2010, 6:19 PM
Subscribers

Description

Hi,

today, Lars Gerigk form the DKFZ Radiology hat a closer look at 3M3.

Unfortunately, the region growing tool, which he already knew (and loved) from former MITK releases, didn't work.

Its incredible sensitive. Also it increases the segmentation range although you already turned around the mouse movement.

Tested on Linux with the 3M3 demo machine at the dkfz.

Event Timeline

[SVN revision 20981]
FIX (#3202): Revert to stable behavior for CT images and perhaps problematic behavior for MRI

I just changed the behavior of the region growing tool to the behavior before revisions 18328/18345, where Daniel and Philipp tried to improve region growing on MR images.

I recognie that the origin of T1540 was also usage of region growing on MR data and that this is still not solved nicely. But with the recent behavior of the tool, region growing was neither useful on CT nor on MR data. Now it is at least useful on CT.

The following ideas sketch a solution:

  • moving the mouse up and down changes the width of the gray value interval which is used to control the region growing algorithm
  • the absolute distance of the mouse position from the initial mouse position is added/subtracted from the interval width (scaled by a fixed factor)
  • to my knowledge, problems only arise when the screen borders are a limit for mouse movement (unlimited screen size would fix the problem)
  • moving the visible mouse cursor back to the initial position after each mouse movement would make infinitely long mouse movements possible (the volume visualization module uses this trick, too)

An implementation problem is that we currently have no means of changing the mouse cursor position from the code location where mouse movement is evaluated for the region growing tool -- this is Qt independent code. However, there is already a class for changing the mouse cursor icon independently of Qt (mitk::ApplicationCursor). This class could be enhanced to also provide and manipulate the absolute mouse cursor position.

Proposed changes to implement described feature.

Now attached the right patch -- previous one was for a different issue.

[SVN revision 20983]
FIX (#3202): enhanced mouse movement area by constantly resetting the mouse position after each move

Region growing feature should be better usable now. I'm closing this issue now until there is new feedback from medical users.