Blender Git Loki

Fix ocean modifier giving different result on Arm than x86

And re-enable the test on macOS Arm.

Ref T78710

Fix T88345: Blender crash on GPUFrameBuffer creation

Misuse of indexes.

`color_attachements` has only color ones while `config` has color and depth.

WM: check missing space-data & constraints in poll functions

Without this, menu search prints many errors in some contexts.

Merge branch 'blender-v2.93-release'

Tests: Disabled physics ocean test on macOS arm64

Test is failing and needs further investigation.

Tests: Remove threshold param from physics ocean mesh test

This was not the reason for the test failure on macOS arm64.

Merge branch 'blender-v2.93-release'

Fix error calling select-camera without a 3D view

Cycles: Avoid unnecessary data updates in viewport

The BlenderSync will do quite a bit of work on every sync_data() call
even if there is nothing changed in the scene. There will be early
outputs done deeper in the call graph, but this is not really enough to
ensure best performance during viewport navigation.

This change makes it so sync_data() is only used when dependency graph
has any update tags: if something changed in the scene the dependency
graph will know it. If nothing changed there will be no IDs tagged for an
update in the dependency graph.

There are two weak parts in the current change:

- With the persistent data there is a special call to ignore the check
of the dependency graph tags. This is more of a safety, because it is
not immediately clear what the correct state of recalc flags is.

- Deletion of objects is detected indirectly, via tags of scene and
collections.

It might not be bad for the first version of the change.

The test file used: {F10117322}

Simply open the file, start viewport render, and navigate the viewport.
On my computer this avoids 0.2sec spend on data_sync() on every
up[date of viewport navigation.

We can do way more granular updates in the future: for example, avoid
heavy objects sync when it is only camera object which changed. This
will need an extended support from the dependency graph API. Doing
nothing if nothing is changed is something we would want to do anyway.

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

Merge branch 'blender-v2.93-release'

Fix error in grease pencil flip color operator

- Used try/except instead of a poll function.
- The error case referenced a non-existent error handling module.

Prefer poll functions over exception handling where possible,
also having an operators logic in a try block isn't good practice
as it can hide more serious errors in the code.

Note that duplicate pencil settings access should be moved into a
utility function. This can be part of a separate cleanup.

PyAPI: allow any vector sequence for poly_3d_calc(..)

Previously only vectors were accepted for the second point argument:
mathutils.interpolate.poly_3d_calc()

Cleanup: use const casts in mathutils API

Cleanup: format

Merge branch 'blender-v2.93-release'

Fix T88026: Repeated switch to rendered viewport crashes

Resolve ownership ambiguity with shared physics pointers.

Previously, LIB_ID_CREATE_NO_MAIN allowed pointer sharing with
the source ID so physics caches can be shared between original and
evaluated data: (Object.soft.shared & Object.rigidbody_object.shared).

This only worked properly for LIB_TAG_COPIED_ON_WRITE ID's,
as LIB_TAG_NO_MAIN can be used in situations where the original ID's
lifetime limited by it's original data.

This commit adds `LIB_ID_COPY_SET_COPIED_ON_WRITE` so ID's only share
memory with original data for ID's evaluated in the depsgraph.

For all other uses, a full copy of physics data is made.

Ref D11228#287094

Cleanup: clang-format

Alembic procedural: fix wrong property used as base for attribute
lookups

We use the schema so that we can access top level attributes as well.
This is already done for polygon meshes and curves, so this only
modifies the behavior for subdivision objects.

Alembic procedural: fix crash accessing invalid ICompoundProperties

Although it is not a pointer, accessing an ICompoundProperty
dereferences a pointer under the hood, so check for validity.

Add comment regarding Fade Inactive Geometry property in overlays popover

Reviewed By: campbellbarton

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