Blender Git Loki

Cycles: Fix amount of rendered samples not being shown while rendering the last tile on CPU

Cycles: Add fast-math safe isnan and isfinite

Currently unused, but might become really handy in the future.

Cycles: Cleanup, spelling

Cycles: Simplify some code in Curve BVH reference fill

makes code slightly shorter and uses idea of const qualifiers.

Cycles: Cleanup, style

Cycles: Fix wrong transparent shadows for motion blur hair

This was a missing bit from b53ce9a.

Cycles: Avoid shadowing in BVH code

Run into some nasty bugs while trying various things here.

Wouldn't mind enabling -Wshadow for Cycles actually..

Cycles: Cleanup, avoid shadowing

Cycles: Correct assert() for cases when there are multiple curves per BVH node

Cycles: Use separate limit for motion primitives for BVH node limits

This way we can have different limits for regular and motion curves
which we'll do in one of the upcoming commits in order to gain some
percents of speedup.

The reasoning here is that motion curves are usually intersecting
lots of others bounding boxes, which makes it inefficient to have
single primitive in the leaf node.

Cycles: Allow up to 4 motion curve primitives per BVH node

This avoids intersection AABB of different curve primitives
which makes it less ray-to-primitive intersections.

This gives about 30% speedup of hair rendering in the barber
shop scenes here. There is still some work to be done on those
files to solve major speed issues on certain frames.

Cycles: Change confusing logic of max leaf size check

Maximal number of elements is supposed to be inclusive. That is what
it was always meant in this file and what @brecht considered still
the case in 6974b69c6172.

In fact, the commit message to that change mentions that we allowed
up to 2 curve primitives per leaf while in fact it was doing up to 1
curve primitive.

Making it real 2 primitives at a max gives about 5% slowdown for the
koro.blend scene. This is a reason why BVHParams.max_curve_leaf_size
was changed to 1 by this change.

Cycles: Cleanup, space prior to semicolon

We don't have that in Blender style, no reason to violate it here.

Cycles: Prepare BVH traversal code to work with multiple curve primitives per node

Cycles: Use dedicated debug passes for traversed nodes and intersection tests

This way it's more clear whether some issue is caused by lots of geometry in
the node or by lots of "transparent" BVH nodes.

Cycles: Make it more clear message why curve motion attribute was removed

Cycles: Fix wrong scaling of traversed instances debug pass

Cycles: Remove more duplicated code in debug passes logic

Cycles: Cleanup, make curve functions private

Not only they don't really follow naming convention (we don't use
camel case) but also was not necessary to keep them in the global
symbol table.

Cycles: Cleanup, indentation within preprocessor