Blender Git Commits

Fix T86208: copy node group button is inconsistent in geometry nodes

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

Fix Cycles NaN assert in random walk SSS due to very small throughput

Now terminate if there are many bounces and the throughput gets so small
that we get precision issues.

Cleanup: minor changes to pose-mode apply visual transform

- Remove use of evaluated poses, instead calculate transformations
into an array which is applied afterwards.

- Only update ID's for poses that have been changed.

Geometry Nodes: Don't create empty components when realizing instances

Previously even if the input goemetry set had no point cloud or no mesh
instances, `geometry_set_realize_instances` would create empty data.
This isn't necessarily bad, but it can complicate things down the line if
there are a bunch of empty components getting passed around.

GPencil: Rename Options panel to Settings

This change is to keep consistency with other panels with the same functionality.

Reviewed by @pablovazquez

Fix T86677: select grouped in node editor crashes without active node

This was reported for geometry nodes, but was true for all nodetrees
(e.g. after deleting the active node). Geometry node trees just made
this more obvious since they start without an active node to begin with.

Fix provided by @lone_noel, thx!

Maniphest Tasks: T86677

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

LibOverride: Outliner: Rename `Add...` to `Make...`.

Also more consistent with the name used for the 3DView operator.

Fix T86701: Geometry nodes Cube and UV Sphere mesh size

The size in the transform matrices was extra, since it is also
passed as an argument to the BMesh operators.

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

Fix T86448 EEVEE: SSRefraction Depth regression

Caused by recent change for contact shadow raytracing. rB4e236326c137

LibOverride: Outliner: Tweak override creation from instancing empty.

Now behavior is similar to the one from 3DView: once override of the
collection is successfuly created, we remove the instancing empty from
the scene.

LibOverride: Fix missing update after an override reset.

We need to brute-force reset IDs with `ID_RECALC_ALL` here...

Fix T86548: Sculpt: Mask by Color tool not working

Caused by {rB2917f550caa9} which renamed the entry in the toolsystem,
but not the corresponding keymap.

Maniphest Tasks: T86548

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

Fix 3DView not updating on some NC_ID notifiers.

Outliner uses a lot `NC_ID | NA_EDITED` e.g., which was not caught by
the View3D editor for update.

Fix missing view3d updates after recent NC_SPACE notifier filters

Since {rB46aa70cb486d}, using `NC_SPACE | ND_SPACE_VIEW3D` as notifier is
restricted to space data as a reference. This was still used though for
RNA updates in other places (namely `rna_camera`, `rna_scene`,
`rna_animviz`), and passing NULL would automatically set the notifier
reference to the owner id. Above commit would happily filter these out,
leading to missing refreshes.

Now use more specific notifiers (in case of animviz a new
`ND_DRAW_ANIMVIZ` was added).

This was reported for Camera background images btw.
Fixes T86670.

Maniphest Tasks: T86670

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

Cleanup: Python code style

Fix race condition when multiple file/asset-lists are running at once

E.g. opening a file with an asset viewer and an Asset Browser could cause one
of them to not load its files properly. That is because there was only one
`wmJob` per scene so when the first one was done it would end the other one(s).
I think this would be a problem in master for the regular File Browser too.

Instead of setting the scene as owner of the job, use the file-list, so that
each file/asset-list gets its own job.

Cleanup: First round of polishing (C++ style, type safety, naming, etc)

* Follow "rule of five"
* Explicit conversion constructors
* Separate inteface declarations from definitions (readability)
* Avoid clang-tidy error
* Naming
....

Merge branch 'master' into ui-asset-view-template

Use custom collection property to pass the asset list to UI lists

`UIList` is designed to use collection properties. I previously did a
whole bunch of hacks to use it without. This isn't acceptable for
master, instead a proper collection property should be provided.

Idea is to let scripts register a custom collection property, that the
asset view template can fill with the latest asset list contents and
pass that to the UI list.

Longer term I'd prefer if the UI code would support other containers,
and not rely so much on RNA and custom/ID properties. I have ideas and
plans for this, but meanwhile, this will do to remove the hacks.

Fix warning about possibly mixed C/C++ linkage of type

Would give a warning:
```
'ED_asset_library_reference_from_enum_value' has C-linkage specified, but
returns incomplete type 'struct AssetLibraryReference' which could be
incompatible with C [-Wreturn-type-c-linkage]
```