Skip to content

Commit

Permalink
ENH: Disable shadows in 3D views by default
Browse files Browse the repository at this point in the history
Shadows feature is not yet mature enough to be enabled by default. It may make rendering darker, not compatible with MIP, and the shader fails to compile if shading (normal computation) is disabled.

See #7560 for MIP incompatibility
See discussion about darkened look at https://discourse.slicer.org/t/are-shadows-enabled-by-default-for-volume-rendering-in-new-previews/34219
  • Loading branch information
lassoan committed Feb 12, 2024
1 parent 59608a2 commit c1abfbf
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion Base/QTGUI/Resources/UI/qSlicerSettingsViewsPanel.ui
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@
<string>Show shadows by default to improve depth perception</string>
</property>
<property name="checked">
<bool>true</bool>
<bool>false</bool>
</property>
</widget>
</item>
Expand All @@ -305,6 +305,9 @@
</item>
<item row="7" column="1">
<widget class="ctkSliderWidget" name="ThreeDAmbientShadowsSizeScaleSlider">
<property name="enabled">
<bool>false</bool>
</property>
<property name="singleStep">
<double>0.010000000000000</double>
</property>
Expand All @@ -331,6 +334,9 @@
</item>
<item row="8" column="1">
<widget class="ctkSliderWidget" name="ThreeDAmbientShadowsVolumeOpacityThresholdSlider">
<property name="enabled">
<bool>false</bool>
</property>
<property name="decimals">
<number>2</number>
</property>
Expand Down

0 comments on commit c1abfbf

Please sign in to comment.