Blender Git Loki

New check_submodules_nv.cmd script.

Modified version of check_submodules.cmd that
checks out and pulls NVIDIA's branch of addons_contrib.

LineArt: In-front UI hints.

LineArt: Floating edge type.

Cleanup: Move PlayAnim frame-cache variables into a struct

Naming was inconsistent and hard to follow.

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

Merge branch 'blender-v2.93-release'

PlayAnim: support limiting the cache by memory instead of frames

Partial fix for T81751 which exposes multiple playback performance
issues. Previously the cache was limited to 30 frames, without a way to
increase the cache for smooth playback with files that are slow to load.

Now the animation plays back smoothly once loaded into cache.

The cache limit from the system preference is used
when the player is launched from Blender.

A new player argument `-c <cache_limit>` was added to support this.

Merge branch 'blender-v2.93-release'

VSE: Fix wrong flag value

`SEQPROP_VIEW_TRANSFORM` value was set incorrectly.
Introduced in 9576612d45dc

Merge branch 'omni-umm' of https://gitlab-master.nvidia.com/omniverse/blender into omniverse

Update addons_contrib in make.bat.

Invoking new update_nv_addons.cmd script in make.bat
to checkout and pull NVIDIA's branch of addons_contrib.
This is currently necessary to always include the latest
UMM addon in the build. This is not an ideal solution
because we pull the latest commits from this branch
on every build, so we need to consider alternatives
to this approach.

Cleanup: spelling

Cleanup: link directly to stackexchange URL

Fix T82002: ENTER does nothing when mouse cursor is over save dialog text field

The `file.execute` operator would early-exit because the mouse wasn't hovering
the file list. Caused by 4ba9d7d71e8f.

Although simpler solutions would have been possible, I decided it's better to
split add a new operator for executing based on the mouse (for double-clicking
files), to separate that from the window level execute operator
(`file.execute`). This allows more control and we can get rid of the implicit
assumption that the keymap would call `file.select` on mouse-press, and
`file.execute` on double-click, for the double-click behavior to work. The cost
is that we execute the file selection & activation logic twice on the
double-click, but that shouldn't be an issue at all.
Also removes the `need_active` property from the `file.execute` operator.
That's ancient and wasn't implemented well anyway.

To be clear, reason this fixes the bug is that `file.execute` works entirely
with the `execute()` callback now and doesn't early-exit based on the mouse
position anymore.

Might trigger warnings about the `need_active` property not being found for
custom keymaps. These can be ignored and the property can safely be removed
from the keymap. I don't expect other keymap breakages.

Merge branch 'blender-v2.93-release'

Fix T86037 EEVEE: SSR option changes render passes result

This was caused by the SSR option resetting the accumulation. But the
render passes were only cleared in the init phase. This means that
when SSR was resetting the `taa_render_sample` the actual renderpasses
would still contains 1 sample. This means the renderpasses were always
divided by the wrong number of samples.

The fix is to clear just before accumulation if the sample is 1.

The fact that it works for motion blur is kind of a blessing. This is because
we check `stl->effects->ssr_was_valid_double_buffer` before resetting the
sampling. So this only happens on the first motion step and does not affect
the rest of the rendering.

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

Remove printfs

Fix T86037 EEVEE: SSR option changes render passes result

This was caused by the SSR option resetting the accumulation. But the
render passes were only cleared in the init phase. This means that
when SSR was resetting the taa_render_sample the actual renderpasses
would still contains 1 sample. This means the renderpasses were always
divided by the wrong number of samples.

The fix is to clear just before accumulation if the sample is 1.

The fact that it works for motion blur is kind of a blessing. This is
because we check stl->effects->ssr_was_valid_double_buffer before
resetting the sampling. So this only happens on the first motion step
and does not affect the rest of the rendering.

Reviewed by: jbakker

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

Fix T86823 Eevee: refraction depth shared between different materials

This was caused by the material grouping system which was missing
a per material uniform update in the opaque case.

Fix T87068 EEVEE: Moire effect with particle hair and subsurface

This artifact was already present in previous version but was hidden
by the faulty SSS scale.

The issue comes from the translucence using the geometric normal
(computed using fragment shader derivative) leading to poor precision at
depth discontinuity.

Replacing using the same geometric normal reconstruction as the ambient
occlusion pass removes most of the issue.