Blender Git Loki

Fix: Incorrect versioning for float IDProperty UI data

The code put the value from the "min" property into the "max"
value. This would have crashed if min was null and max wasn't.

Fix T91088: Assigning custom property value in python resets UI data

Assigning a new value to an IDProperty with the Python API would free
the entire contents of the existing property, which unfortunately
happened to include the UI data. The fix is to extract the UI data from
the existing property before freeing its contents. An alternative
would be adding another argument to `IDP_FreePropertyContent_ex`, but
this solution is clearer and doesn't increase complexity elsewhere.

Fix Cycles X wrong denoising prefilter default for final render

It should be Accurate.

Fix T91084: Missing versioning for object pose bone property UI data

Objects also have a list of "bone" pose channels embedded directly.
These properties are user visible, so their UI data should be versioned.

adaptive_cloth: AdaptiveMesh: force split for sewing

Option to split the opposite edge even if it does not meet the size
criterion, with the option, it ensures that no sewing edge is missed
when an edge is split (it can be missed if the opposite edge doesn't
meet the size criterion).

Cleanup: Reduce variable scope

adaptive_cloth: AdaptiveMesh: mark sewing edge verts as preserve

Mark all verts attached to sewing edge(s) as preserve, this ensures
that no sewing edge(s) are removed which would otherwise lead to
results are not in line with what the artist would want.

Use `ghc::filesystem` as platform compatible replacement for `std::filesystem`

Previous commit merged the `temp-cpp-ghc-filesystem` branch into this, now we
can actually use this replacment. This way the branch can be compiled on macOS.

Merge branch 'temp-cpp-ghc-filesystem' into temp-asset-browser-catalogs

Dependencies: Drop-in replacement for std::filesystem (in extern/)

Introduces the ghc::filesystem library as a (leight weight, header
only) drop-in replacement for std::filesystem. The latter can't be
used on the minimum macOS version we support. Once that changes, we can
switch to using std::filesystem directly.

Part of T90379.

* Add ghc::filesystem to extern/ as platform compatible replacement for
std::filesystem.
* Add Windows specific code to avoid issues with including Windows.h.
* Add blender::bli::filesystem as alias for ghc::filesystem. That should make
the ghc -> std transition easier later on since we can just change this
alias. Also makes it easy to switch between the two at compile time.

Merge branch 'master' into temp-cpp-file-system

Merge branch 'master' into temp-asset-browser-catalogs

Fix bad memory access in OIDN when sample count pass present in Cycles X

Such configuration reads combined pass into RGB array. The alpha channel
was not properly ignored in the pass accessor in this case.

Test file: F10357795

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

Fix: Output int for precision in UI data as_dict method

This is stored internally and used as an integer, so there is no need
to convert it to a float for "as_dict". This was just an oversight.

Cleanup: pass value by ref in draw_color_management

adaptive_cloth: AdaptiveMesh: ensure edge between sewing edges

While trying to create the sewing edges, ensure that the vert in
question is between 2 or more edges that are between sewing edges.

Also ensure that the opposite is between sewing edges.

Fix T88433: no greaspencil depsgraph evaluation with certain drivers

When the same stroke was used as a driver variable, this could make this
stroke already tagged as built in the course of building driver
variables (via `build_gpencil`), but then important stuff from
`build_object_data_geometry_datablock` could be missed later on (because
both of these funtions use `checkIsBuiltAndTag`). Most importantly,
setting up operations such as GEOMETRY_EVAL would be skipped entirely.

`build_object_data_geometry_datablock` seems to cover greasepencil just
fine (does the same as `build_gpencil` and more). Proposed solution is to
remove `build_gpencil` entirely. In `build_id` it would then also call
`build_object_data_geometry_datablock` for `ID_GD` IDs. Now the covered
types that _call_ `build_object_data_geometry_datablock` match exactly
to what is covered _inside_ `build_object_data_geometry_datablock`.

Think this "duplication" of functionality was just overseen in
rB66da2f537ae8 [`build_gpencil` existed long before and said commit made
greasepencil a real object with geometry and such].

thx @JacquesLucke for additional input!

Maniphest Tasks: T88433

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

Fix missing tiles with GPU render and OIDN denoiser in Cycles X

Was noticed when testing upcoming tiled rendering with small tiles
sizes which stresses data transfers more than single tile rendering.

What happens is OIDN denoiser uses `copy_to_device()` using default
queue, which is not guaranteed to be synchronized before next call to
`copy_from_device()` happening via queue.

What we really want in the OIDN denoiser is synchronous data transfer
which is easiest to be achieved by creating a queue and synchronizing
it.

The issue is reported by Alaska in D12309#320253

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

Fix missing result when cancelling render with denoiser in Cycles X

OIDN will stop demoising when user cancel is requested. This might
leave render buffers in a partially updated state, so the tile write
logic will wrongly assume denoised result exists.

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

Merge branch 'master' into asset-greasepencil