Page MenuHomePhabricator

Replace DiffusionImaging PlanarFigureMapper3D by common PlanarFigureVtkMapper3D
Closed, ResolvedPublic

Description

There is a hard-coded replacement in DiffusionImaging for several types of planar figures regarding the 3D mapper. This leads to errornous behavior and rendering of these planar figures types if the DiffusionImaging module is built. Additionally, the DiffusionImaging planar figure 3d mapper is quite rudimentary, incomplete and contains several bugs.

Solution: Remove the hard-coded mapper replacement and the DiffusionImaging planar figure mapper itself. Set the "planarfigure.3drendering" property to true if needed.

Event Timeline

User kislinsk has pushed new remote branch:

bug-17513-PlanarFigure3DRenderingWhenDiffusionImagingIsEnabled

@Peter

Would you please check if the changes are okay for you. I guess you will want to set the "planarfigure.3drendering" to true somewhere in your code.

[93cedf]: Merge branch 'bug-17513-PlanarFigure3DRenderingWhenDiffusionImagingIsE

Merged commits:

2014-03-17 11:17:20 Stefan Kislinskiy [aca955]
Default to enabled planar figure 3D rendering for diffusion imaging views.


2014-03-14 12:18:24 Stefan Kislinskiy [b1e1e5]
Removed PlanarFigureMapper3D.

Use PlanarFigureVtkMapper3D by setting "planarfigure.3drendering" property to true.