diff --git a/test-renderers/expected/renderers/tone_mapping-Clamp-0.5-wgpu.png b/test-renderers/expected/renderers/tone_mapping-Clamp-0.5-wgpu.png deleted file mode 100644 index 805f4881b..000000000 Binary files a/test-renderers/expected/renderers/tone_mapping-Clamp-0.5-wgpu.png and /dev/null differ diff --git a/test-renderers/expected/renderers/tone_mapping-Reinhard-0.5-wgpu.png b/test-renderers/expected/renderers/tone_mapping-Reinhard-0.5-wgpu.png index 74a3ff329..04bb9e468 100644 Binary files a/test-renderers/expected/renderers/tone_mapping-Reinhard-0.5-wgpu.png and b/test-renderers/expected/renderers/tone_mapping-Reinhard-0.5-wgpu.png differ diff --git a/test-renderers/expected/renderers/tone_mapping-Reinhard-2.0-wgpu.png b/test-renderers/expected/renderers/tone_mapping-Reinhard-2.0-wgpu.png index 0105abded..966d2ea7b 100644 Binary files a/test-renderers/expected/renderers/tone_mapping-Reinhard-2.0-wgpu.png and b/test-renderers/expected/renderers/tone_mapping-Reinhard-2.0-wgpu.png differ diff --git a/test-renderers/src/test_cases.rs b/test-renderers/src/test_cases.rs index 641f44b5d..bd94bc1b7 100644 --- a/test-renderers/src/test_cases.rs +++ b/test-renderers/src/test_cases.rs @@ -937,17 +937,11 @@ async fn tone_mapping(mut context: RenderTestContext, (tmo, exposure): (ToneMapp context.universe(), ); - // TODO: Ideally there would be at most 1 difference but there are large-enough-to-be-visible - // ones for some reason. These appeared, not when working on the tone mapping algorithm, but - // when changing PackedLight’s rounding behavior. - // Also, there is a notable ray/wgpu difference concentrated in the middle range of Reinhard. + // TODO: There is a notable ray/wgpu difference concentrated in the middle range of Reinhard. // So, there are probably bugs lurking here. + // (The thresholds here are *not* for those difference but other pixel differences.) context - .render_comparison_test( - Threshold::new([(20, 200), (10, 500), (3, 2000), (2, 20000)]), - scene, - Overlays::NONE, - ) + .render_comparison_test(Threshold::new([(10, 100), (3, 500)]), scene, Overlays::NONE) .await; }