Blender Git Loki

Geometry nodes: Output curve normals from the normal node

The code is basically the same as rB4d51af68adb273.

Geometry Nodes: Curve tangent node

This node outputs the direction vector, or tangent of a curve at every
control point. For poly splines this is simply the evaluated tangents,
so it all works very simply. For Bezier splines it uses the tangent at
the evaluated point corresponding to each control point, and NURBS
are interpereted as poly splines built from their control points.

Internally the node is called "Input Tangent" to simplify using it for
mesh tangents as well in the future like the "Normal" node.

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

Cleanup: Clang-tidy warnings

UI: Style drag-drop indicators as tooltips

Currently, the drop indicator colors are hardcoded to white text on semi-transparent black background.
This patch makes the drop indicator use the tooltip theme settings, as they serve a similar purpose.

{F10530482, size=full}

All built-in themes seem to work well and got improved readability.

Reviewed By: HooglyBoogly

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

Fix: Add versioning for geometry nodes attribute input toggle

rB8e21d528cab98 neglected to add versioning to add the new
"use_attribute" and "attribute_name" properties to the modifier input
list. Though they are added if the modifier's interface is updated, that
doesn't happen when the file is loaded, so patch adds them manually.
Another solution would be calling `MOD_nodes_update_interface`,
but that would require including the modifiers module.

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

Cleanup: Move curve to mesh node implementation to blenkernel

I plan to use this for curve object data conversion to mesh in D12533,
and possibly for the implicit curve to mesh conversion in the curve
and text object modifier stack in the future.

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

Asset Catalogs: prevent writing of empty files

Asset Catalogs: nicer temporary filename when writing

Cleanup: mention `UUID_STRING_LEN` in comment

Mention that `UUID_STRING_LEN` exists in the documentation of
`BLI_uuid_format()`.

No functional changes.

Cleanup: clarify `AssetCatalogService::write_to_disk()`

No functional changes.

Fix: Incorrect versioning for float IDProperty default array

The versioning has to have different behavior for when the old default
value had a float type rather than a double type. Also use memcpy
rather than dupalloc, since it's a bit clearer.

Cleanup: else-after-return

Functions: make asserts more correct

It is valid to e.g. copy construct an integer in the same place,
because it is a trivial type. It does not work for types like std::string.

This fixes a crash reported in D12584 where it would copy a buffer
into itself. We should probably also avoid doing this copy alltogether
but that can be done separately.

Cleanup: mention `UUID_STRING_LEN` in comment

Mention that `UUID_STRING_LEN` exists in the documentation of
`BLI_uuid_format()`.

No functional changes.

Asset Catalogs: defer writing of catalog definitions

Instead of writing the catalog definition file (CDF) immediately after
creating a new catalog, wait until the `write_to_disk(...)` function is
called.

As a result, it's now also possible to create an `AssetCatalogService`
without passing it an asset library root path (for example when creating
catalogs in a not-yet-saved blend file and without having an asset
library opened/configured/selected).

Fix T91534: GPencil interpolate Sequence fails if stroke has 0 points

In some cases the stroke has 0 points and this must be skipped in the interpolation.

Port UI code to use the new catalog UUIDs

GPencil: Curvature support for length modifier

Extend the stroke following an approximated circluar/helical curve.

This can be used as an effect for lineart or on its own as helix generator.

Reviewed By: Sebastian Parborg (zeddb), Hans Goudey (HooglyBoogly), YimingWu (NicksBest), Antonio Vazquez (antoniov), Henrik Dick (weasel)

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

# ?????????????? '#' ????????????????
# ?????????
#
# ???? master
# ????????? 'origin/master' ???
#
# ???????
# ??? source/blender/blenkernel/BKE_gpencil_geom.h
# ??? source/blender/blenkernel/intern/gpencil_geom.cc
# ??? source/blender/gpencil_modifiers/intern/MOD_gpencillength.c
# ??? source/blender/makesdna/DNA_gpencil_modifier_defaults.h
# ??? source/blender/makesdna/DNA_gpencil_modifier_types.h
# ??? source/blender/makesrna/intern/rna_gpencil_modifier.c
#

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

Fix T87801: Eevee ambient occlusion is incorrect on M1 macMini

The issue was caused by `textureSize()` returning the size of the level 0
even when the min texture level is higher than 0.

Using a uniform to pass the correct size fixes the issue.

This issue also affected the downsampling of radiance for reflections and
refractions.

This does not affect anything other than the recusive downsampling shaders.