Wrong recursion stop case.
attiju opened this issue · comments
Jules Attivissimo commented
Shouldn't you stop recursion when the number of primitives in current node is equal or bellow maxPrimsInNode ?
In the current version, end == start + 1
then the for loop is useless.
I suppose you wanted the if condition to be if (nPrimitives <= maxPrimsInNode) { ... }
PBrtJ/src/org/pbrt/accelerators/BVHAccel.java
Line 317 in 65bab7c