Blender Git Commits

Cleanup: remove duplicate variables

Previous code made it seem as if the original edit-object
was re-created after separating.

Edit Armature: match separate selection with meshes

- Existing objects remain selected.
- The content of the armatures is unselected.

Fix separating bones on hidden layers

Fix T72310: Separate bones creates empty armatures

Armatures that didn't contain any selected bones were still
creating armatures.

Add the ability to create internal springs to the cloth sim

This can be used to make closed surfaces behave more like a soft body.

Reviewed By: Jacques Lucke

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

Fix T72006: Clay brush with size pressure corrupts mesh

When size pressure is enabled, the radius may not be enough to sample
any vertices to calculate the area normal. This lead to a vector 0 plane
and a division by 0 when calculating the signed distance to the plane,
which produces a NaN that was being written to the vertex coordinates.

Reviewed By: brecht

Maniphest Tasks: T72006

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

Fix T72054: Sculpt Mode crash when using Relax Mesh Filter with Dyntopo enabled

This commit fixes 3 bugs:

- Fix the crash reported in T72054. The BMesh elem table and the vd.no was null, so we now ensure that the table exists before running any sculpt tool in dyntopo. The relax function also uses vd.fno in case that vd.no is not available.
- Fix missing updates of the bounding boxes when running the mesh filter. This can be optimized by running the updates only when the filter finishes. Without this, it is impossible to sculpt the user modifies the mesh too much with the filter.
- Fix incorrect solution of relax vertex when using EEVEE. Relaxing the mesh requires the updated normals after each iteration. This was done by the PBVH rendering code, but when running EEVEE it was using incorrect normals. Now normals are updated after each iteration.

Reviewed By: brecht

Maniphest Tasks: T72054

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

Fix T72092: Clay Strips Brush stroke crash with Brush Detail Size

Some other areas in the brush code outside sculpt mode assume that
pressure is multiplied directly on top of the initial size. This patch
calculates the pixel radius correctly using the brush size from sculpt
mode to get the dyntopo detail size.
When the new brush input system is in place, all these values will come
directly from the brush input code with all the custom curves applied
per brush, so all paint modes will have a correct brush behavior and all
this sculpt mode specific code won't be necessary.

Reviewed By: jbakker

Maniphest Tasks: T72092

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

Fix Mask Brush gradient artifacts

The old mask brush implementation was adding the brush value to the
previous vertex mask value and clamping the result. This leads to
visible artifacts in the mask gradient as the value approaches 0 or 1,
so it was not possible to paint a smooth mask with this brush.
Now we are also multiplying by the previous mask value before clamping,
fixing all those gradient artifacts.

Reviewed By: jbakker

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

Cleanup: Redundant variable assignment to self

Was triggering strict compiler warning.

Cycles: Add OptiX acceleration structure compaction

This adds compaction support for OptiX acceleration structures, which reduces the device memory footprint in a post step after building. Depending on the scene this can reduce the amount of used device memory quite a bit and even improve performance (smaller acceleration structure improves cache usage). It's only enabled for background renders to make acceleration structure builds fast in viewport.

Also fixes a bug in the memory management for OptiX acceleration structures: These were held in a dynamic vector of 'device_memory' instances and used the mem_alloc/mem_free functions. However, those keep track of memory instances in the 'cuda_mem_map' via pointers to 'device_memory' (which works fine everywhere else since those are never copied/moved). But in the case of the vector, it may decide to reallocate at some point, which invalidates those pointers and would result in some nasty accesses to invalid memory. So it is not actually safe to move a 'device_memory' object and therefore this removes the move operator overloads again.

Reviewed By: brecht

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

Fix T72145: Object.update_from_editmode crashes with empties

Also corrects the return value, where objects that don't have an
edit-mode returned true, making it seem as if object data was flushed.

Gizmo: add view aligned gizmo for shear

Fix T71878: Cycles crash with adaptive subdivision and empty mesh

Fix T72282: Cycles OpenCL error after recent math node changes

Fix T71480: crash with Eevee image texture node and clipping

Fix T72269: crash with Eevee and normal maps after recent math node changes

Reusing math node functions for normal map node caused problems.

Fix T72240: GLSL error with some old NVIDIA drivers, causing crash at startup

Fix T72252: crash running ocean bake in background mode

Fix T72064, T72158: weight paint shows X/Y/Z symmetry options that don't work

Only X-mirror is supported still. While we should support all these in the
future, the UI should match the current supported functionality.

Ref T65872