Blender Git Commits

Cleanup: KnifeTool: Replace glPolygonOffset by GPU_polygon_offset

Cleanup: GPU: Encapsulate clipDistances

This also remove some non functionnal state changes that were left
from legacy code.

UI: Use property split for custom property edit popover

The panel looks out of place with the rest of Blender's UI and the
text is cropped. With property split turned on and a few smaller
tweaks these issues are fixed.

| Before | After |
|{F8700181}|{F8700183}|

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

Simulation: extract node tree parsing code to separate file

Simulation: separate code from node tree parsing and solver

Fix T78959: Show current frame indicator when interface is locked

When the playhead drawing moved to an overlay, a check was added to keep
it from drawing with a locked interface. This is necessary for some overlays,
but not this one, so this removes the check, making it the responsibility of
the editor.

A context function is added to make that check easier in the future.

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

Transform: use GHASH_ITER when restoring customdata

Cleanup: Removed incorrect `// namespace DEG` comment

No functional changes.

Fluid: Numpy support for Mantaflow build system

Adjusted the fluid build system so that plugins that depend on numpy can be compiled as well.

Note that in this commit numpy support is still disabled. It can be enabled by re-running the Mantaflow update script with USE_NUMPY=1 and enabling WITH_MANTA_NUMPY in extern/mantaflow/CMakeLists.txt. This will happen in a future commit.

Fluid: Cleanup build system for extern mantaflow

No longer including unused dependencies. Should numpy IO be needed at some point, the Manta source update script can be configured so that the required dependencies are included again.

Refactor: move ParticleFunction to separate file

Cleanup: avoid static initialization order issues when accessing CPPTypes

Instead of depending on static initialization order of globals use
static variables within functions. Those are initialized on first use.
This is every so slighly less efficient, but avoids a full class of problems.

Simulation: move initial simulation code from bf_blenkernel to bf_simulation

I removed bf_blenkernel from `nodes/CMakeLists.txt` again (added it yesterday),
because now this was causing me unresolved symbol errors... Without it, cmake
seems to link the libraries bf_simulation, bf_blenkernel and bf_nodes in the right
order. Not sure if that is just luck or if it is guaranteed.

It was possible to fix the issue by using cmakes `LINK_INTERFACE_MULTIPLICITY`,
but that is probably bad style.

Fix T77867: Link Duplicate Object crashes during batch creation

When using link duplicated objects it could happen that one object is
calculating the GPUBuffers and the second object is marking these
buffers invalid. This introduces threading issues.

This patch fixes this by combining the surface and surface per material
batches. Most likely the surface per material batches are used and when
requested you will most likely need the surface batch for the depth
tests and overlays.

During tests it slightly improves performance as batches aren't thrown
away without using it.

After this patch we can add a quick path for meshes with one material
and two materials.

Alternative approaches that have been checked:
- sync extraction per object: reduced performance to much (-15%)
({D8292})
- post checks: reduced the threading issues, but didn't solve it.
- separating preparation and execution of the extraction ({D8312})

Reviewed By: Cl�ment Foucault

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

Cleanup: silence unused variable warning

Reviewed By: fclem

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

BLI: add StringRefNull.c_str() method

This should be used whenever you rely on the fact, that the
returned pointer points to the beginning of a null-terminated array.

Simulation: use better api for adding and removing simulation states

Cleanup: avoid warning about redundant access specifier

No functional changes.

Nodes: move some code from blenkernel directory to nodes

This also introduces the `blender::nodes` namespace. Eventually,
we want to move most/all of the node implementation files into
this namespace.

The reason for this file-move is that the code fits much better
into the `nodes` directory than in the `blenkernel` directory.

Cleanup: consistent ordering for scene argument to UV functions