From 4a6b9456aa863fbb06bb523cf49a1a9776bcf74a Mon Sep 17 00:00:00 2001 From: Tom Power Date: Wed, 3 Dec 2025 16:30:16 -0800 Subject: [PATCH] Set default headlight intensity to zero if there are no physical lights in the scene. PiperOrigin-RevId: 839962392 Change-Id: I086d8135604982c42381a9b49cec53773c7fa362 --- src/experimental/filament/filament/scene_view.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/experimental/filament/filament/scene_view.cc b/src/experimental/filament/filament/scene_view.cc index 575a3bf0..4977bf47 100644 --- a/src/experimental/filament/filament/scene_view.cc +++ b/src/experimental/filament/filament/scene_view.cc @@ -310,7 +310,9 @@ void SceneView::PrepareLights() { // dealing with a "classic renderer" scene. In this case, let's add a // default environment light and set the light intensity ourselves. if (total_light_intensity == 0.0f) { - constexpr float kHeadlightIntensityCandela = 10'000.f; + // Headlight is not required for Filament and often confusing, disable it by + // default. + constexpr float kHeadlightIntensityCandela = 0.f; constexpr float kTotalSceneLightIntensityCandela = 100'000.f; constexpr float kFallbackEnvironmentLightIntensityCandela = 10'000.f;