Skip to content

Commit

Permalink
Constrain max value test further for unbounded
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
jalberse committed Sep 30, 2023
1 parent ffdc498 commit 3c49322
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/pbrt/util/spectrum_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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);

Expand Down

0 comments on commit 3c49322

Please sign in to comment.