View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001798 | Slicer4 | Core: Base Code | public | 2012-03-09 10:19 | 2017-06-07 23:27 |
Reporter | spujol | Assigned To | finetjul | ||
Priority | normal | Severity | feature | Reproducibility | sometimes |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.0.1 | ||||
Target Version | Slicer 4.3.0 | Fixed in Version | Slicer 4.3.0 | ||
Summary | 0001798: Backface culling in Models Module | ||||
Description | The Backface Culling option is checked by default in the Models module. | ||||
Additional Information | Mac Snow Leopard | ||||
Tags | No tags attached. | ||||
Why do you don't want it checked by default ? |
|
When several models are simultaneously displayed, backface culling can make the 3D rendering a little bit confusing to the user. |
|
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) |
|
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. |
|
Depth peeling sorts the order in which models/polygons are rendered on screen 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. |
|
Is there a performance hit for using depth peeling? |
|
yes, on the CPU and GPU (multiple pass rendering), but it might not be too noticeable (have to be tested) |
|
How about introducing this as an option, like back face culling? |
|
Agree. |
|
Fixed in r22159: depth peeling is a new option for the 3D view |
|
Closing resolved issues that have not been updated in more than 3 months |
|
Slicer: 2145-support-for-installing-extension-from-file 92e3f03b 2013-07-13 04:13:16 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 |
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 |