Page MenuHomePhabricator
Create Task
Maniphest T18578

Surface 2D Mapper and Geometry (fun)
Closed, ResolvedPublic
Actions

Description

The new SurfaceVtkMapper2D does not work corretly, if the object has a modified geometry (e.g. BoundingObject of the ImageCropper).

We tried to modify UpdateVtkTransform, but it did not work yet.

Related Objects

Event Timeline

• kilgus added projects: MITK, Bugzilla.Dec 22 2014, 6:03 PM
• kilgus added a subscriber: • kilgus.Jan 8 2015, 1:15 PM

I wrote a core change request and will push the solution now.

• git-user added a subscriber: • git-user.Jan 8 2015, 1:15 PM

User kilgus has pushed new remote branch:

bug-18578-Surface2DMapperGeometry
• kilgus added a comment.Jan 8 2015, 1:16 PM

The solution is explained in the core change request and in the documentation of the SurfaceVtkMapper2D.

• kilgus added a comment.Jan 8 2015, 2:22 PM

I discovered that the PointSetVtkMapper2D has the same issues. There it is solved with a manual transformation of each point. This can be done via the UpdateVtkTransform method of VtkMapper if a vtkAssembly would be used. However, the vtkTextActors have to be replaced by some 3D actor in this case. Christoph and I will have a look at this next Wednesday.

• kilgus added a comment.Jan 16 2015, 10:42 AM

[68193a]: Merge branch 'bug-18578-AngleFix'

Merged commits:

2015-01-16 10:25:27 Thomas Kilgus [25425b]
Changing Ascii code to keyboard degree sign.

2015-01-08 14:07:35 Thomas Kilgus [b69f26]
Cosmetic fixes and docu. 3D mapper uses smartpointers now.

2015-01-08 13:01:29 Thomas Kilgus [4d5415]
Delete old tests which are now in new test suite.

2015-01-08 12:31:16 Thomas Kilgus [60f434]
Improved documentation.

2015-01-08 11:57:53 Thomas Kilgus [57400d]
Added new test suite for the surface 2D mapper. It replaces all the former tests. New helper class for argv/argc in rendering test helper.

2015-01-08 11:56:50 Thomas Kilgus [a95432]
Fixed geometry issue of the 2D mapper.

2015-01-07 13:54:05 Thomas Kilgus [f55c5e]
Cosmetic fixes.

• kilgus added a comment.Jan 16 2015, 10:56 AM

[e72c59]: Merge branch 'bug-18578-FixWarning'

Merged commits:

2015-01-16 10:48:41 Thomas Kilgus [3da38f]
COMP: removed unused member.

• kilgus added a comment.Jan 16 2015, 11:16 AM

[90b332]: Merge branch 'bug-18578-FixWarning'

Merged commits:

2015-01-16 11:08:22 Thomas Kilgus [3fa666]
COMP: mitkSurfaceVtkMapper2DTest is only build if the cmake flag is set.

• kilgus added a comment.Jan 16 2015, 12:20 PM

[84dd55]: Merge branch 'bug-18578-FixWarning'

Merged commits:

2015-01-16 12:11:49 Thomas Kilgus [d536ad]
COMP: Fixed UpdateVtkTransform.

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