Blender Git Loki

Geometry Nodes: Improve speed of boolean node, use multi-input socket

This commit improves the performance of the node by up to 40% in some
cases when there are only two input meshes, mainly by skipping the
conversion to and from BMesh.

When there are more than two input meshes (note the distinction from
"Geometries", a geometry set can have many mesh instances), the
performance is actually worse, since boolean currently always does
self intersection in that case. Theoretically this could be improved
in the boolean code, or another option is automatically realizing
instances for each input geometry set.

Another improvement is using multi-input sockets for the inputs, which
removes the need to have a separate boolean node for every operation,
which can hopefully simplify some node trees.

The changes necessary for transforms in `mesh_boolean_convert.cc` are
somewhat subtle; they come from the fact that the collecting the
geometry set instances already gives transforms in the local space
of the modifier object. There is also a very small amount of cleanup
to those lines, using `float4x4::identity()`.

This commit also fixes T87078, where overlapping difference meshes
makes the operation not work, though I haven't investigated why.

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

Cleanup/Refactor: Use flags instead of bool to configure the snap gizmo

This simplifies the addition of future improvements.

Also make it more practical to expose as a parameter of gizmo for Python.

Cleanup: Remove unused enum value in editmesh_knife.c

Fix compilation error: "too many arguments to function"

Seem on linux.

Caused by rB6ec463a4b754bf69baf94ba6b3683655f6834ccd

Cleanup/Refactor: Transform Orientation: Use 'orient_index' instead 'orient_type'

`orient_index` is a more comprehensive value as it reveals both the type and index.

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

Cleanup: typo fix in comment

No functional changes.

Fix T87065: Deformed Lattice undeforms after setting keyframe

Handle Lattice object the same way as Mesh objects. This is mostly to
execute the `object->data = data_eval;` line, which ensures that the
evaluated mesh is assigned to the evaluated object, and thus prevents
the lattice from un-deforming.

BLI: rename resource collector to resource scope

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

Curve: Remove 'CU_2D' flag used for nurbs

This fixes T86440

As the CU_2D flag is set for nurbs, a Curve can have 2D nurbs mixed with 3D.

But the UI does not allow this mixing. It updates all nurbs to 2D or 3D when set.

So remove this specific flag for nurbs.

This may break old files, since 2D curves with mixed 3D are now set as 3D.

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

Cleanup: use bool instead of int

Cleanup: Remove unused code

There was a note about reusing this for "texture nodes", but that will
probably not be implemented in this way anyway.

Cleanup: move node_exec.c to c++

Doing this, because it might make it easier to replace the implementation
of `bNodeInstanceHash`.

Nodes: separate node name and display name in bNodeTreePath

Previously, `node_name` was rarely actually a name of a node. It is set
correctly as node name in `ED_node_tree_push`. However, later on it
was overwritten by the name of an id data block in `node_draw_space`.

Now, the node_name stays the name of the "parent" node. Whereas
display_name is the name that will be displayed in the breadcrumbs.

With this change, the `node_name` can be used to reconstruct the
actual path from the root node tree to the currently visible tree.

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

Cleanup: spelling

Geometry Nodes: add socket value logging capability

The node tree evaluator now calls a callback for every used socket with
its corresponding value(s). Right now the callback does nothing.
However, we can use it to collect attribute name hints, socket values
for debugging or data that will be displayed in the spreadsheet.

The main difficulty here was to also call the callback for sockets in
nodes that are not directly executed (such as group nodes, muted
nodes and reroutes).

No functional changes are expected.

BLI: return early when copying empty array

BLI: add call_safe method for FunctionRef

This is useful to avoid nullity checks in some places.

LineArt: Remove resampling settings.

This can be done with an other GP modifier already.

LineArt: Remove "soft selection" option.

After some back and forth with the GP module and some artists, this
option was deemed not that useful. The use case was considered too
obscure so we'll remove it.

It is still posible to have this functionality by using the vertex
weight modiifers or manually clamping the weights.

Fix "unused variable" warning when compiling without nanovdb