mmp / pbrt-v3

Source code for pbrt, the renderer described in the third edition of "Physically Based Rendering: From Theory To Implementation", by Matt Pharr, Wenzel Jakob, and Greg Humphreys.
http://pbrt.org
BSD 2-Clause "Simplified" License
4.86k stars 1.18k forks source link

Is refraction implemented correctly in v3 BDPT? #332

Closed rainbow-app closed 8 months ago

rainbow-app commented 8 months ago

Hello.

I'm thinking of borrowing some pbrt code. However, I have a suspicion that refraction implementation is faulty. Is my suspicion a result of my ignorance, or is the refraction really wrong?

Thanks.

rainbow-app commented 8 months ago

Ok, it's my ignorance. Read the textbook, then looked at code: the direction is tracked in mode in si. Sorry.