Blender Git Loki

Fix T86932: Curve: pick shortest path missing update of active point drawing

For curves, we need to tag the curve ID_RECALC_COPY_ON_WRITE for batch
cache update (same as in {rB24b2fe50f3ec}).

Maniphest Tasks: T86932

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

Fix T84623: Curve/Surface force not working in normal direction

Tweaking e.g. a field strength would then not use the curve/surface
normal anymore [but the object center instead].

If a curve has a forcefield with effector shape Curve (in code its shape
is PFIELD_SHAPE_SURFACE then), it wil get a SurfaceModifier.

Changing properties will free the SurfaceModifierData's bvhtree and mesh
And these dont get copied along when doing the CoW copy, these are
explicitly set to NULL. So this was also failing for meshes, not just
curves.

Without the mesh & bvhtree though, get_effector_data() will not set the
EffectorData's normal correctly (it is closest_point_on_surface() which
does this). And without the right EffectorData's normal, the effector
will of course work unexpected.

Going in and out of editmode made this work because that goes down this
route:
- BKE_object_handle_data_update
- BKE_displist_make_curveTypes
- do_makeDispListCurveTypes
- curve_calc_modifiers_post
-- BKE_mesh_new_nomain_from_curve_displist
-- we then have our desired updated mesh from the curve
-- this will also call the SurfaceModifiers deformVerts [which - given we
have a valid mesh - will update the bvhtree properly]

Also note that _animating_ the effector actually works, (have not done
the deep dive why this works, assume the curve geometry is updated in
this case)

So, now just carefully tag the curve ID_RECALC_GEOMETRY in
rna_FieldSettings_update for this specific case.

Maniphest Tasks: T84623

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

Cleanup: Resolve moving a temporary object prevents copy elision warning

Cleanup: Explicitly mark method as overrided

Fix T86175: Missing viewport update when toggling HDRI preview

Caused by rB0f95f51361d7.

Similar to T85515, T84717, T85543 and their related fixes.
In this case, add missing shading notifier as in rB9274bd457a25.

Reviewed By: brecht, lichtwerk

Maniphest Tasks: T86175

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

Merge branch 'master' into temp-compositor-single-threaded-operation

Suppress compiler warning.

Merge branch 'master' into temp-compositor-single-threaded-operation

Geometry Nodes Curves: Small cleanup, remove debugging code

Geometry Nodes Curves: Generate faces in Curve to Mesh node

Fix T48167: Mesh select random factor is not accurate

Selecting random percentage would not reliably guarantee the precise
percentage of selected elements now randomize an index array instead
and use the precise number of elements from this array.

Note that this change has only been made to edit-mesh
and should be applied to all random selection operators.

Merge branch 'master' into temp_bmesh_multires

Forgot to increase array size for data transfer modifier struct.

Geometry Nodes Curves: Further clean up forward differencing function

Cleanup: Vectorize bezier forward differencing

It's a bit strange no one had done this already. This also solves the
problem of `BKE_curve_forward_diff_bezier` always affecting one
more points than you asked for, which was causing another problem.

Fix T87187: Area.type & ui_type empty area access fails

Reading Area.type & ui_type printed an error from `pyrna_enum_to_py`
and returned an empty string.

As empty is a valid value, include it in the property enum list.

Disable delta compression, restore curve key packing.

Delta compression is now disabled since it was using the chunks system
which was removed.

Add UI settings to better control BVH refits and delta compression

Fix BVH::num_refits not being incremented.

This was lost in the OptiX multi device support.

Revert OptiX vertex copy optimization.

This was failing for the multiple devices case as it is based on the
device_pointer of the device_memory, which is not really a pointer when
multiple devices are used, but an index in the actual device pointers
map.

Revert for now, until we have a nice solution.