Page MenuHomePhabricator
Create Task
Maniphest T3263

Autocropping segmentations produces offset (due to padding)
Closed, ResolvedPublic
Actions

Description

Reproduction of this bug:

  • start ExtApp with segmentation view, load Pic3D
  • create a quick segmentation covering a couple of slices (e.g. using interpolation)
  • create a surface model using the data manager context-menu
  • you can observe that surface and binary image align in the sagittal view
  • now choose "autocrop" from the data manager context-menu
  • scroll a slice or two in sagittal
  • by now the surface and the binary image misalign (see screenshot)

I suspect that this has to do with the recent change in autocropping: a padding is applied to create "closed" surfaces from cropped images. This padding is not taken into account when setting the cropped image's geometry.

Ingmar, I'm assinging this to you because I think you changed the padding, right?

Please wait with fixing this bug until I finished #3163 which involves quite some code restructuring.

.1 (ITK 3.16.0 VTK 5.4.0 Qt 4.5.3 MITK n-a) (Not for use in diagnosis or treatment of patients).png (1×1 px, 277 KB)

Related Objects
Search...

Event Timeline

maleike added projects: MITK, Bugzilla.Jan 29 2010, 11:36 AM
nolden added a subscriber: nolden.Feb 24 2010, 4:09 PM

T3163 is fixed now

maleike added a subscriber: maleike.Mar 17 2010, 3:45 PM

#3384 is most probably a duplicate. I'm leaving this bug here open because it has a description of how to reproduce the bug.

• santos added a subscriber: • santos.Mar 17 2010, 5:51 PM

No, it is not a duplicate.

gergel added a subscriber: gergel.Mar 18 2010, 10:25 AM

In QmitkSegmentationPostProcessing::IncreaseCroppedImageSize the origin of the
new segmentation image should also be translated according to the added offset.

This should/could solve the problem.

muellerm added a subscriber: muellerm.Mar 22 2010, 7:35 PM

[SVN revision 21849]
FIX (#3263): Added a file chooser widget (a button + lineedit + filedialog)

muellerm added a comment.Mar 22 2010, 10:37 PM

[SVN revision 21850]
COMP (#3263): Fixed EndoTracking dependency for landmarktracking

maleike added a comment.Mar 23 2010, 9:31 AM

(In reply to comment #5)

[SVN revision 21849]
FIX (#3263): Added a file chooser widget (a button + lineedit + filedialog)

wrong bug id?

gergel added a comment.Mar 24 2010, 12:17 PM

[SVN revision 21869]
FIX (#3263): fixed offset problem due to padding, geometry is now also translated

kislinsk changed the status of subtask T3384: image cropper crops wrong region from Invalid to Spite.Jun 27 2018, 1:32 PM
kislinsk changed the status of subtask T3384: image cropper crops wrong region from Spite to Invalid.Jun 27 2018, 1:36 PM
MITK · MIC · IMSY · DKFZ