Blender Git Loki

UI: Support setting operator properties for `UILayout.operator_menu_enum()`

`UILayout.operator_menu_enum()` now returns the operator properties, just like
`UILayout.operator()`. This makes it possible to set options for the operator
displayed in the menu. In C it can be done through the new
`uiItemMenuEnumFullO()` or `uiItemMenuEnumFullO_ptr()`.

It's reasonable to have this, probably just a small thing never bothered to
add. D10912 could use it, the following comment can be addressed now too:
https://developer.blender.org/diffusion/B/browse/master/source/blender/editors/space_nla/nla_buttons.c$583-586

Only reset the node view when links are cancelled.

Fix CPU rendering and OptiX denoiser in Cycles X

Make sure copying of temporary render buffers to the device
happens as part of the denoiser queue.

Initially thought this will be fixed by some more global changes
related to multi-GPU support, but:

- It is a simple and clear change.
- It brings multi-device support to a working state, which makes
it easier to verify changes.
- Multi-device is supported at a higher level, and it could still
be useful to support current usecases of DeviceDenoiser (where
input render buffers are allocated on a different device).

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

Cycles X: Ensure OptiX denoiser setup and usage uses the same stream

Avoids possible access of denoiser non-initialized state without
explicit sync.

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

Cycles X: Remove zero_to_device from RenderBuffers::reset()

In all the current usages of RenderBuffers::reset the zero was
actually redundant.

This change makes it easier to allocate temporary render buffer
and copy data to it as a part of a specific stream without zero
(which happens in the default stream) conflicting.

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

Cleanup: Win32 Window Creation

This is just some cleanup of the Win32 window creation code. After
CreateWindowExW() this patch uses some early exits to replace some
potentially confusing if blocks. No functional changes.

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

Reviewed by Ray Molenkamp

Vulkan: Retrieve platform information.

Retrieve device platform information for `GPUPlatformGlobal`.
As vulkan needs a physical device it was moved from the GPUBackend
constructor to an init method that is called when an initial context
is created.

Still need to find out how to extract the driver type.

Fix Cycles hair render error on GPU after recent changes

Volumes primitive needs to be part of traceable primitives.

Cleanup: Remove unused function in lineart_cpu.c

Fix T70615: Cycles ignores BSDF inputs when nodes are optimized

When compiling BSDF nodes, we only assing stack space to the normal and
tangent inputs if they are linked. However, it could be that the
ConstantFolder removed the link, so checking if there is a link fails
to take this into account.

To fix this, added a flag to ShaderInput to keep track of whether a
constant was folded into the input, and use it as well to verify that
the socket is linked when assigning stack space.

Reviewed By: brecht

Maniphest Tasks: T70615

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

Cleanup: add `const` to local variables and function parameters

Add `const` to some variables that don't need to be mutable.

No functional changes.

Cleanup: typo fix

Add a comma.

No functional changes.

Cleanup: reduce amount of conditional code

Use `return;` in `if`-block, making it possible to un-indent the entire
`else` block.

No functional changes.

Cleanup: remove unused `#include`

Remove unused `#include <stdio.h>`.

No functional changes.

Cleanup: `CTX_wm_asset_handle`, move assignment of return param

Move the assignment to return parameter `*r_is_valid` to always be just
above the corresponding `return` statement. This makes it easier to ensure
that it is always set, and to see that it's only set once.

No functional changes.

Merge remote-tracking branch 'origin/master' into asset-browser-poselib

Clean up a few warnings

LineArt: Filtering intersection lines using mask numbers

Mask value works just like transparency mask.

You are able to select intersection lines inside a
collection or, between collections.

Reviewed By: Sebastian Parborg (zeddb)

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

Cycles X: Fix crash using CPU and OptiX denoiser in viewport

Not sure why it is only visible now, there was a buffer overrun since
the initial CPU Render + OptiX denoise support commit by the looks of
it.

There now seems to be a synchronization issue between queues, leading
to only partial buffer update in the viewport. Investigating this now.

Fix T89484: NLA "Remove Empty Animation Data" missing Outliner refresh

Similar to rBb4530deec478, just add appropriate notifier.

Maniphest Tasks: T89484

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