Page MenuHomePhabricator

Rename all rendering properties
Closed, WontfixPublic

Description

First, we should discuss some naming issues. For instance, how do we call "binary images"? Maybe "Segmentation"?

According to the following name scheme:
-Image Rendering.Opacity
-Image Rendering.Window Level.Maximum
-Image Rendering.Color
-Surface Rendering.Color
-Volume Rendering.XYZ...

Affacted classes (not exhaustive):
-mitkImageVtkMapper2D
-mitkGeometry2DDataVtkMapper3D
-...

Event Timeline

Very easy to fix. Very good for beginners.

We created a list of all Rendering properties and going to discuss to upcoming changes in the next meeting.

PropertyList

See also:

http://docs.mitk.org/nightly-qt4/PropertiesPage.html
http://docs.mitk.org/nightly-qt4/SegmentationPropertiesPage.html
http://docs.mitk.org/nightly-qt4/PlanarPropertiesPage.html
http://docs.mitk.org/nightly-qt4/DiffusionImagingPropertiesPage.html

And T12140 which concerns itself with the automatic documentation of properties.

Personally I dislike the spaces. I would prefer:
-ImageRendering.Opacity
-ImageRendering.WindowLevel.Maximum
-ImageRendering.Color
-SurfaceRendering.Color
-VolumeRendering.XYZ...

Until #14257 is solved, we should not work on this bug.

Rendering properties naming has to be discussed

We cannot solve this until we have a solution for scene file loading of older properties.

kislinsk claimed this task.
kislinsk added a subscriber: kislinsk.
This task was automatically closed because it wasn't updated at least since July 2016 (over 2 years). Please re-open this task if you think that it is still relevant. This most probably means that you will resolve it.