Blender Git Loki

Merge branch 'master' into temp-gpencil-bezier-stroke-type

GPencil: Add Bezier support to Mirror modifier

Revert incorrect change.

Merge branch 'master' into cycles_procedural_api

Compositor: Merge SingleThreadedOperation with WriteBufferOperation.

Single threaded operations always creates a buffer around and write
buffer operation would sample that buffer one pixel at a time. This
refactor would push the samples in one go to the output buffer by
merging the write buffer and single threaded operation.

Less code complexity, less needed CPU cycles and less memory would be
needed.

GPencil: Add Bezier support to Thickness modifier

GPencil: Add Bezier support to Hook modifier

Cleanup comments

Cycles: Fix CUDA kernel compilation error

Was broken with aa_samples removal.

Comment out the offending cryptomatte code. It needs to be looked
closely and redone anyway. Was probably wrong with adaptive sampling.

Cycles: Remove unused aa_samples from KernelIntegrator

Was needed for CMJ to work, but we no longer have CMJ and we are only
interested in patters which are progressive.

Cycles: Remove conditional AA samples update

It was only needed for CMJ to work.

The baking is unclear why it was needed, since the feature set which
depends on number of AA samples is either unsupported for bake (such
as adaptive sampling) or is not needed since CMJ removal.

Cycles: Remove resolution divider from KernelCamera

Cycles: Remove residue of FILM_CONVERT task

It is taken care of in GPUDisplay.

Cycles: Remove redundant type prefix

Types are defined within CPUKernels, so the CPU prefix is redundant.

Cycles: Use typed function for convert_to_half_float kernel

Cycles: Remove film conversion to byte buffer

The motivation is that such conversion is faster to be performed as
a fragment shader.

For platforms and configurations which can not use fragment shader
an alternative would be to implement it in the GPUDisplay level.

If this functionality is really something what a lot of Cycles
adopters are needing, we can reconsider this. Until then is better
to keep an API small and clean.

Cycles: Remove dead code from Session

The code is not reachable, not testable, and unlikely to be needed for
the final implementation. Remove it to make ongoing development and
maintenance cleaner.

For the reference we can always consult code from the previous Cycles
releases.

There are still some `#if 0` in the `session.cpp` but they are not in
a way and can be removed later on.

Cycles: Remove unused update_scene() function

Cycles: Remove or comment out end-of-life code

The goal is to make API clean, and stop worrying about keeping it
compilable. The most interesting bits of implementation which we
will need to reference for the new implementation are `#if 0`-ed.

The code which is either migrated or which is known to be not needed
is removed.

Summary:

- Remove `DeviceTask` and `Device::task_foo`
It is replaced with either queue or CPUKernels.

- Comment out NLM denoiser.
Can probably remove it. But lets have a quick meeting about it.

- Remove old OpenImageDenoiser implementation.
It is now implemented in integrator/denoiser.

- Commented out render/denoiser. It is to be ported to the new
denoiser implementation in inetgrator/. Also, if NLM is removed then
feature-parity is not really possibly at this time.

- Comment out device_denoiser.

Cycles: Remove path_trace kernel

It is implemented using PathTrace and PathTraceWork.