chunky-dev / chunky

A path tracer to create realistic images of your Minecraft worlds.
https://chunky-dev.github.io/docs
GNU General Public License v3.0
643 stars 76 forks source link

Workaround for some potential drawbacks of ray branching #1620

Closed JustinTimeCuber closed 1 year ago

JustinTimeCuber commented 1 year ago

Don't branch on the first few SPP to avoid some potential performance-related issues (such as pointed out here on Discord) based on the square root of the branch count (arbitrary, but seems to work fine)

Examples: If BC=5, sequence will be 1, 1, 1, 2, 5, 5, 5, ... If BC=10, sequence will be 1, 1, 1, 1, 6, 10, 10, 10, ... If BC=20, sequence will be 1, 1, 1, 1, 1, 15, 20, 20, 20, ...

Also, this sets the default branch count to 10 instead of 5 (since there are now fewer issues with that.)