Page MenuHomePhabricator
Create Task
Maniphest T2337

Easy: Remove PolyDataGLMapper2D
Closed, ResolvedPublic
Actions

Description

mitk::PoinData is no longer used by any part of MITK. It should be removed together with all associated classes (mappers etc.), in particular

  • mitk::PointDataVtkMapper3D
  • mitk::PolyDataGLMapper2D
NOTE: mitk::PolyDataGLMapper2D could possibly be used for rendering other (arbitrary) VTK based structures as well, but this is not done in MITK. However, it should be checked if mitk::PolyDataGLMapper2D should be retained for other purposes.

Related Objects

Event Timeline

fangerau added projects: MITK, Bugzilla.Jul 30 2009, 4:22 PM
gergel added a subscriber: gergel.Feb 2 2011, 2:42 PM

[SVN revision 29487]
FIX (#2337): removed deprecated mitk::PointData and mitk::PointDataVtkMapper3D

gergel added a comment.Feb 2 2011, 2:48 PM

[SVN revision 29488]
COMP (#2337): removed deprecated include

gergel added a comment.Feb 2 2011, 3:07 PM

mitk::PolyDataGLMapper2D was not removed.

If this is considered important it should be specified in an extra bug.

• kilgus added a subscriber: • kilgus.Oct 26 2011, 2:05 PM

Todo: Remove PolyDataGLMapper2D.h and PolyDataGLMapper2D.cpp and all the correspoding includes. Very simple cosmetic change.

Should be redirected to someone new in rendering for a first feeling of success ;).

• kilgus added a comment.Oct 26 2011, 2:11 PM

Markus and I decided to review this Mapper again. This mapper is not using mitk::PointData at all and could be usefull for rendering vtkPolyData in 2D.

• kilgus added a comment.Oct 8 2012, 5:39 PM

Still todo: Make a review.

• kilgus added a comment.May 22 2013, 1:52 PM

Markus and I talked about this. This is a simple example but the documentation is bad and it contains "FIXME" comments. Additionally, it is more or less a duplicate of the SurfaceGLMapper2D and hence should be removed.

• kilgus added a comment.Oct 9 2013, 2:34 PM

This bug is not very important, but pretty old and hence it should be fixed.

Todo: Remove PolyDataGLMapper2D.cpp + .h

• fetzer added a subscriber: • fetzer.Dec 9 2013, 3:51 PM

Current release is finished. Reseting target milestone...

• wintersa added a subscriber: • wintersa.Jan 8 2014, 3:51 PM

New remote branch pushed: bug-2337-RemovePolyDataGLMapper2D

• wintersa added a comment.Jan 15 2014, 1:36 PM

[e5e3c0]: Merge branch 'bug-2337-RemovePolyDataGLMapper2D'

Merged commits:

2014-01-15 13:33:32 Adrian Winterstein [7a8809]
Changed deprecated notice to release tag.

2014-01-08 15:43:58 Adrian Winterstein [293184]
Set PolyDataGLMapper2D deprecated.

2014-01-08 15:29:49 Adrian Winterstein [9531b8]
Removed useless includes of PolyDataGLMapper2D.

• wintersa added a comment.Jan 15 2014, 1:38 PM

Class is documented as deprecated, now. Can be removed after release 2014-03.

• git-user added a subscriber: • git-user.Apr 9 2014, 2:28 PM

User wintersa has pushed new remote branch:

bug-2337-RemovePolyDataGLMapper2D_ActualRemoving
• wintersa added a comment.Apr 9 2014, 4:39 PM

[aeacab]: Merge branch 'bug-2337-RemovePolyDataGLMapper2D_ActualRemoving'

Merged commits:

2014-04-09 13:45:37 Adrian Winterstein [ef0476]
Class PolyDataGLMapper2D removed.

kislinsk merged a task: T15337: mitkPolyDataGLMapper2D either deprecated or should be ported to VTK.Aug 1 2016, 10:47 PM
kislinsk edited projects, added MITK (2016-11); removed MITK.
MITK · MIC · IMSY · DKFZ