Page MenuHomePhabricator

Migrate Region Grower from IpPic to Image Accessors
Closed, ResolvedPublic

Description

Unsigned short replaced unsigned char as the default segmentation type. The region grower no longer works and should be rewritten using image accessors.

Event Timeline

User petersej has pushed new remote branch:

bug-19447-Rewrite-Region-Grower

We have a good contour, but it's not being displayed. m_FeedbackContour of FeedbackContourTool stores the contour, but RequestUpdateAll() doesn't do anything. m_FeedbackContourNode's data pointer has m_UpdateBoundingBox set to false (compare with SetRegionTool, where this is true). Could this be the reason? How can we change this property?

[7c8b52]: Merge branch 'bug-19447-Rewrite-Region-Grower'

Merged commits:

2016-02-24 16:45:49 Jens Petersen [5f0c32]
Fixed documentation for region grower


2016-02-24 15:41:30 Jens Petersen [655d7c]
Remove two unused dependencies


2016-02-24 15:37:19 Jens Petersen [2f99da]
Region grower no longer uses ipSegmentation

For now, no behaviour is implemented for when the user clicks inside a segmentation.


2016-02-17 16:53:50 Jens Petersen [96637a]
Fix growing part of region growing

Still need to rewrite what happens when users click inside existing segmentation


2016-02-10 18:28:05 Jens Petersen [716e70]
Minor changes


2016-01-27 17:47:18 Jens Petersen [0d53b3]
Extract contour, but is not displayed


2016-01-13 17:22:45 Jens Petersen [dcbef7]
Have working segmentation smoothing

Figure out how to return segmentation from AccessByITK
Rewrite so that thresholds holds lower and upper threshold


2015-12-09 17:37:49 Jens Petersen [b2fd05]
Have working region growing

Need to smooth result and fix contour extraction (what to do with multiple contours)
Need to adapt OnMouseMoved method
Need to check that writing back to image works


2015-12-02 18:06:02 Jens Petersen [642053]
Have first iteration of region grower, still crash


2015-12-01 11:55:54 Jens Petersen [00adcd]
Start implementing region grower

[063b84]: Merge branch 'bug-19447-Rewrite-Region-Grower'

Merged commits:

2016-02-25 12:09:11 Jens Petersen [47db39]
COMP: Initialize itk::Index with double braces

[ac0920]: Merge branch 'bug-19447-Rewrite-Region-Grower'

Merged commits:

2016-02-25 12:54:48 Jens Petersen [94ce3c]
COMP: Move itk::Index init to constructor body

[1eb0f9]: Merge branch 'bug-19447-Rewrite-Region-Grower'

Merged commits:

2016-02-25 14:56:56 Jens Petersen [88aff1]
COMP: Initialize itk::Index with single braces

[f7b600]: Merge branch 'bug-19447-Rewrite-Region-Grower'

Merged commits:

2016-02-25 15:11:00 Jens Petersen [5ddb62]
COMP: Not initialize seed point&thresholds at all