search

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

Ambiguity in SamleCatmullRom2D #283

Closed abrassel closed 4 years ago

abrassel commented 4 years ago

On page 825 of PBRT, Third Edition, the book writes "The rest of [SampleCatmulRom2D] is identical to SampleCatmullRom except that every access to values[i] is replaced by interpolate(values, i)." In SampleCatmullRom, there aren't actually any calls to values[i].

mmp commented 4 years ago

Good catch--thanks! I'll have that added to the errata and fixed in the online edition when the next update is pushed online later today.