Blender Git Commits

Fix quaternion compatibility function

Use closest quaternion instead of only checking w sign flipping,
which didn't catch all cases.

T

Clip Editor: Fix uninitialized plane track grabbing points

Was happening when there was an image assigned to the track.

Another pair programming session with Clement.

Clip Editor: Fix visualization of plane track image

Pair programming with Clement.

Clip Editor: Allow opening and creating images from Plane Track panel

Differential Revision: https://developer.blender.org/D6170

Fix T64655: Quad view toggle conflicts on macOS

Cmd-Alt-Q is a system shortcut on macOS, use Ctrl-Alt-Q.

BLI_task: Add pooled threaded index range iterator, Take II.

This code allows to push a set of different operations all based on
iterations over a range of indices, and then process them all at once
over multiple threads.

This commit also adds unit tests for both old un-pooled, and new pooled
task_parallel_range family of functions, as well as some basic
performances tests.

This is mainly interesting for relatively low amount of individual
tasks, as expected.

E.g. performance tests on a 32 threads machine, for a set of 10
different tasks, shows following improvements when using pooled version
instead of ten sequential calls to BLI_task_parallel_range():

| Num Items | Sequential | Pooled | Speed-up |
| --------- | ---------- | ------- | -------- |
| 10K | 365 us | 138 us | 2.5 x |
| 100K | 877 us | 530 us | 1.66 x |
| 1000K | 5521 us | 4625 us | 1.25 x |

Differential Revision: https://developer.blender.org/D6189

Note: Compared to previous commit yesterday, this reworks atomic handling in
parallel iter code, and fixes a dummy double-free bug.

Now we should only use the two critical values for synchronization from
atomic calls results, which is the proper way to do things.

Reading a value after an atomic operation does not guarantee you will
get the latest value in all cases (especially on Windows release builds
it seems).

Fix T71909: Any keyframed GPLayer property is not updated when render animation

By error, the original datablock was used while rendering. Actually, only while the user is drawing the original data must be used because when we tested, the time the system uses to copy the datablock created a very bad "lag" feeling while drawing. Maybe the lag was half second only, but it ruined the pencil feeling.

I talked some time ago with Sergey about that and we decided use this approach.

Now, only the original datablock is used while the user is "moving" the pen, but not in any other situation.

Thanks @sergey for help me with this bug and sorry for thinking it was a depsgraph issue.

Fix T71860: No versioning for drivers in Mapping node.

The new Mapping node was missing versioning code for drivers.
This patch refactors existing code and add versioning for drivers.

Reviewed By: Sergey Sharybin, Bastien Montagne

Differential Revision: https://developer.blender.org/D6302

Fix segfault when polling `MESH_OT_paint_mask_extract`

`CTX_data_active_object(C)` returns `NULL` when there is no active object,
and this was not tested for in the code.

Fix memory leak, closing a window didn't free gestures

Exposed by test file in T71718

Buildbot: Increaser codesign timelimit

It was possible that it would exceed when signing all the DLLs if the machine
is busy with some background tasks or when internet is slow.

Fix T71806: BMElem.copy_from(other) failed

Don't modify the Python reference when copying custom-data.

BMesh: support copying & freeing layers by type

fix: T71718 Collada: importer does not release intermediate fcurves when importing matrix animations

Bevel Modifier: Reset default width to 0.1 from 1.0

Revert "BLI_task: Add pooled threaded index range iterator."

This reverts commit f9028a3be1f77c01edca44a68894e2ba9d9cfb14.

This is giving weird heisenbug crash on only Windows release builds...
Reverting until we understand to issue.

Revert "Cleanup: Unused variable in release build mode"

This reverts commit e0cada951982093453a91b80342ce20c4f421fc8.

UI: Allow label for Template-ID (respecting property split layout)

Adds a `text` parameter to `bpy.types.uiLayout.template_ID()` which
causes a label to be added, as usual. Adding the label also makes the
template respect the `bpy.types.uiLayout.use_property_split` option.

Also fixes wrong layout being used in the template-ID, although I think
that didn't cause issues in practice.

Sergey requested this for usage in the Movie Clip Editor.

Fix potential access to deleted memory in OptiX kernel loading code

Calling "OptiXDevice::load_kernels" multiple times would call "optixPipelineDestroy" on a pipeline
pointer that may have already been deleted previously (since the PIP_SHADER_EVAL pipeline is only
created conditionally).
This change also avoids a CUDA kernel reload every time this is called. The CUDA kernels are
precompiled and don't change, so there is no need to reload them every time.

UI: Widget Text Cursor Color

Adds a theme setting to specify color of widget text insertion cursor (caret).

Differential Revision: https://developer.blender.org/D6024

Reviewed by Campbell Barton