View Issue Details

IDProjectCategoryView StatusLast Update
0001798Slicer4Core: Base Codepublic2017-06-07 23:27
Reporterspujol Assigned Tofinetjul  
PrioritynormalSeverityfeatureReproducibilitysometimes
Status closedResolutionfixed 
Product VersionSlicer 4.0.1 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.3.0 
Summary0001798: Backface culling in Models Module
Description

The Backface Culling option is checked by default in the Models module.

Additional Information

Mac Snow Leopard

TagsNo tags attached.

Activities

finetjul

finetjul

2012-03-09 10:29

administrator   ~0003827

Why do you don't want it checked by default ?
It is Slicer3's default behavior.

spujol

spujol

2012-03-12 07:43

developer   ~0003836

When several models are simultaneously displayed, backface culling can make the 3D rendering a little bit confusing to the user.

finetjul

finetjul

2012-03-12 16:44

administrator   ~0003847

Last edited: 2012-03-12 16:44

Shall we add an option to force all backface culling to be ignored ?

I think what's confusing is more the order of rendering of the models than the backface culling. I would suggest to support Depth Peeling instead (slow)

kikinis

kikinis

2012-03-13 10:49

developer   ~0003851

the default of backface culling was introduced for two reasons: 1. Improved performance for scenes that are large compared to the available render performance. 2. fewer visible artefacts when models are transparent.
btw: what is depth peeling?

finetjul

finetjul

2012-03-13 11:00

administrator   ~0003853

Depth peeling sorts the order in which models/polygons are rendered on screen
http://www.vtk.org/Wiki/VTK/Examples/Cxx/Visualization/CorrectlyRenderingTranslucentGeometry

Currently, depending on the view points, some models seem to appear in front of others while they are supposed to be behind. This is because it depends of the order in which it has been added into the scene, not its 3d position with regard to the camera.

kikinis

kikinis

2012-03-13 11:30

developer   ~0003854

Is there a performance hit for using depth peeling?

finetjul

finetjul

2012-03-13 11:57

administrator   ~0003855

Last edited: 2012-03-13 12:00

yes, on the CPU and GPU (multiple pass rendering), but it might not be too noticeable (have to be tested)

kikinis

kikinis

2012-03-13 12:31

developer   ~0003857

How about introducing this as an option, like back face culling?

finetjul

finetjul

2012-03-13 12:38

administrator   ~0003858

Agree.
One difference, it is not a model property but a 3D view property (all objects or none for the depth peeling).

finetjul

finetjul

2013-07-09 05:49

administrator   ~0008922

Fixed in r22159: depth peeling is a new option for the 3D view

jcfr

jcfr

2014-03-06 05:18

administrator   ~0011105

Closing resolved issues that have not been updated in more than 3 months

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 92e3f03b

2013-07-13 04:13:16

jcfr

Details Diff
BUG: Fix vtkMRMLCameraDisplayableManagerTest1

Following r22159 fixing issue 0001798, the "useDepthPeeling" view node
property has been introduced, this commit updates the mrml input file
used by the test with this new property.

Related to 0001798

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22183 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/DisplayableManager/Testing/Data/vtkMRMLCameraDisplayableManagerTest1.mrml Diff File

Issue History

Date Modified Username Field Change
2012-03-09 10:19 spujol New Issue
2012-03-09 10:19 spujol Status new => assigned
2012-03-09 10:19 spujol Assigned To => finetjul
2012-03-09 10:29 finetjul Note Added: 0003827
2012-03-09 10:30 finetjul Status assigned => feedback
2012-03-12 07:43 spujol Note Added: 0003836
2012-03-12 16:34 finetjul Summary Models Module => Backface culling in Models Module
2012-03-12 16:44 finetjul Note Added: 0003847
2012-03-12 16:44 finetjul Note Edited: 0003847
2012-03-12 16:45 finetjul Severity minor => feature
2012-03-12 16:45 finetjul Reproducibility always => sometimes
2012-03-12 16:45 finetjul Assigned To finetjul =>
2012-03-12 16:45 finetjul Status feedback => new
2012-03-13 10:49 kikinis Note Added: 0003851
2012-03-13 11:00 finetjul Note Added: 0003853
2012-03-13 11:30 kikinis Note Added: 0003854
2012-03-13 11:57 finetjul Note Added: 0003855
2012-03-13 12:00 finetjul Note Edited: 0003855
2012-03-13 12:31 kikinis Note Added: 0003857
2012-03-13 12:38 finetjul Note Added: 0003858
2012-08-20 07:34 nicole Assigned To => finetjul
2012-08-20 07:34 nicole Status new => assigned
2012-08-20 07:34 nicole Target Version => Slicer 4.3.0
2013-07-09 05:49 finetjul Note Added: 0008922
2013-07-09 05:49 finetjul Status assigned => resolved
2013-07-09 05:49 finetjul Fixed in Version => Slicer 4.3.0
2013-07-09 05:49 finetjul Resolution open => fixed
2014-03-06 05:18 jcfr Note Added: 0011105
2014-03-06 05:20 jcfr Status resolved => closed
2017-06-07 23:27 jcfr Changeset attached => Slicer 2145-support-for-installing-extension-from-file 92e3f03b