Page MenuHomePhabricator
Create Task
Maniphest T2627

mitk::SurfaceVtkWriter: Add option for disabling transformation of poly data
Closed, WontfixPublic
Actions

Description

mitk::SurfaceVtkWriter writes mitk::Surfaces as vtkPolyData in any supported VTK file format (via template mechanism). For writing the data, the Geometry / transform contained in mitk::Surface is applied to the individual points, since VTK file formats do not support storing of transforms.

In some cases, it is not desirable that the poly data are transformed automatically, e.g. when the Geometry/transform is stored separately.

The writer class should be extended with an option to enable/disable the application of the transform.

Event Timeline

seitelm added projects: MITK, Bugzilla.Oct 8 2009, 10:08 AM
seitelm added a subscriber: seitelm.Oct 8 2009, 10:14 AM

ChangeRequest

ChangeRequest.txt735 BDownload

maleike added a subscriber: maleike.Oct 8 2009, 10:35 AM

This could be easily tested in a unit test. Could you open a second bug for extending the surface writer test case? Could be done by anybody during bug squashing.

seitelm added a comment.Oct 8 2009, 2:51 PM

[SVN revision 19379]
ENH (#2627): Added option for disabling transformation of polydata via Geometry transform before writing

seitelm added a comment.Oct 8 2009, 2:53 PM

Added required feature with r19379.

TODO: write unit test for new methods. Adding "party" keyword.

nolden added a subscriber: nolden.Oct 16 2013, 1:46 PM

@Michael: this was discussed yesterday. Maybe you can close this after you have examined the current implementations for PointSet and Surface.

goetzm closed this task as Wontfix.Apr 24 2018, 10:04 AM
maleike added a comment.Apr 24 2018, 2:08 PM

@goetzm: just out of curiosity: this looks like something where a bug + fix is available. Could you share why this is closed 'Wontfix'?

goetzm added a comment.Apr 24 2018, 2:18 PM

This is a quite old bug (From 2013). If i remember right, it has been already solved by other bugs over time, as also indicated by marcos last comment.

I cleaned by Task-list, found this bug, I'm pretty sure it is resolved / no longer needed / or no more valid. So non of the reasons had a 100% fit, and i decided to go for Wonfix, although i'm happy to change the status is needed.

kislinsk reopened this task as Open.Jun 27 2018, 1:05 PM
kislinsk removed goetzm as the assignee of this task.
kislinsk added a project: Bulk Edit.
kislinsk added a subscriber: goetzm.
kislinsk closed this task as Wontfix.Jun 27 2018, 1:10 PM
kislinsk removed a project: Bulk Edit.
MITK · MIC · IMSY · DKFZ