From 3c4932275fdf48ff7960e48b5c78f051d1ab10bd Mon Sep 17 00:00:00 2001 From: John Alberse Date: Sat, 30 Sep 2023 15:38:16 -0500 Subject: [PATCH] Constrain max value test further for unbounded Removes a 10x multiplier on the maximum value in the MaxValue test for RgbUnboundedSpectrum. The 10x multiplier makes the check unnecesarily lax; the evaluations on the various lambdas below it should pass without the multiplier applied. --- src/pbrt/util/spectrum_test.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pbrt/util/spectrum_test.cpp b/src/pbrt/util/spectrum_test.cpp index cc4129596..ff10dfa89 100644 --- a/src/pbrt/util/spectrum_test.cpp +++ b/src/pbrt/util/spectrum_test.cpp @@ -109,7 +109,7 @@ TEST(Spectrum, MaxValue) { EXPECT_LE(sr(lambda), m); RGBUnboundedSpectrum su(*RGBColorSpace::sRGB, 10 * rgb); - m = su.MaxValue() * 1.00001f * 10.f; + m = su.MaxValue() * 1.00001f; for (Float lambda = 360; lambda < 830; lambda += .92) EXPECT_LE(su(lambda), m);