Blender Git Loki

Revert "Enhanced stats/reports for blendfile reading."

This change crashes library linking operators, related tests and probably more.

This reverts commit f8d219dfd4c31a918e33cb715472d91a5cd3fd51.

Ref D11583

Geometry Nodes: Join curve attributes when realizing instances

Previously the code assumed that curve instances had no attributes.
This is true when the data came from curve objects, which don't support
attributes currently, but it isn't necessarily true when retrieving curves
from evaluated geometry sets.

Cleanup: Refactor spline copying functions

Make the virtual functions protected and simpler, so that the logic is
better contained in the base class's implementation. Also introduce a
`copy_without_attributes` method to be used for realizing instances.

Fix deadlock with shrinkwrap and other modifiers

More code that needs task isolation. Encountered in sprite fright production
file.

Ref D11603

Enhanced stats/reports for blendfile reading.

Add direct user feedback (as a warning report) to user when recursive
resync of overrides was needed.

And some timing (as CLOG logs) about main readfile process steps.

This is essentially adding a new BlendFileReadReport structure that wraps
BKE_reports list, and adds some extra info (some timing, some info about
overrides and (recursive) resync, etc.).

Minor cleanup to previous commit introducing `BLI_math_time`.

Forgot to address latest review comments, sorry for the noise.`:wq

Add initial `BLI_math_time` with a 'seconds decompose' function.

Allows to decompose a given amount of seconds into a random set of
days/hours/minutes/seconds/milliseconds values.

Also add matching test.

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

Fix T89360: Eevee transforms geometry when using "High Quality Normals"

Offset missed in rB44d2479dc36f

Cleanup: split BKE_object_batch_cache_dirty_tag

Create a more specialized function `BKE_object_data_batch_cache_dirty_tag`

Cleanup: deduplicate free code

It is more appropriate that `depths` is freed in `ED_view3d_depths_free`.

Draw: Added testcases for hair refine shaders.

Fix T89347: Edit-mesh make planar faces doesn't update face normals

Invalid face normals were used for tessellation.

Fix T89345: Edit-mesh decimate doesn't update face normals

Invalid face normals were used for tessellation.

Cleanup: Use more clear visibility tag function name

No functional changes. Just makes it clear this is not an immediate
update, and will make an upcoming change more localized.

Fix T89196: Depsgraph use-after-free after scene switching undo

Delay depsgraph visibility update tagging until it is known that
graph relations are up to date, and until it is known that the graph
is actually needed to be evaluated.

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

Fix error in context menu when built without freestyle

Cleanup: remove unused menu VIEW3D_MT_edit_mesh_edges_data

8aa17c5b12d734332c0af62a110524c4e523fb64 missed removing the
menu definition when removing from the UI.

Cleanup: rename playhead in the UI and code

Current convention is not to use this term, use "current frame",
and "timeline frame" in render.c as this is the argument passed in.

3D View: adjust order of planes for occlusion check

Move far plane last since it's the least likely to intersect edges.

OptiX: select BVH build options from Scene params

Currently, the OptiX BVH build options are selected based on whether
we are in background mode (final renders) or not (viewport renders).
In background mode, the BVH is built for fast path tracing and low
memory footprint, while in viewport, it is built for fast updates.

However, on platforms without OpenGL support, the background flag is
always set to true and prevents using fast BVH builds in the viewport.

Now, the BVH options derive from the Scene BVH settings:
* if BVH is static, a fast to trace BVH is built
* if BVH is dynamic, a fast to update BVH is built

Reviewed By: #cycles, brecht

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