Blender Git Loki

Cycles: add back light intersection for multiple importance sampling

* Intersect closest kernel now intersects with point, spot and area lights
in addition to scene geometry. This could later be optimized by adding
lights to the BVH.
* If such a light is hit, the new shade_light kernel will be executed
which will evaluate the light shader and write to the render buffer,
then continue with intersect_closest.
* The shade_background kernel handles intersection and shader evaluation
for distant lights.

Cleanup: Compiler warnings with COM_TM_NOTHREAD active.

Cleanup: document `FileSelectAssetLibraryUID::type`

No functional changes.

Lineart: remove TODO in object_ops.c

LibOverride: Add a new operation to Outliner to enforce resync of hierarchies.

This is basically done by ignoring override operations from old override
affecting ID pointer properties, when the new (destination) one is not
NULL.

Fix T86501: New object added to overridden collection doesn't show up in linking file on Resync.

This is more of a work-around actually, since there is no real way to
fix the issue in a fully automated and consistent way, it is caused by
older blender files being saved with 'broken' overrides.

WARNING: This cannot ensure that some purposedly edited/overridden ID
pointer properties won't be lost in the process.

LineArt: Fix wrong return value for cheking LRT_GPENCIL_IS_BAKING

LineArt: change ED_ to MOD_

Remove the remaining WITH_LINEART cmake variables

Initial USD Import open sourcing.

Cherry-picked commit of Tangent Animation's
USD importer from branch ta-usd-import in
https://github.com/tangent-opensource/blender.git.

Fixed compile errors. Registering USD plugins.

USD importer stage traversal options.

Added new stage traversal options to filter prim import
based on purpose (guide, proxy and render) and whether
to traverse instances with instance proxies. Also modified
the stage traversal logic to avoid a small memory leak due to
creating a reader for the pseudo root which is never used or
deleted.

USD importer: custom normals shading bug.

Removed call to BKE_mesh_calc_edges() after setting
UVs, as this function is already called after setting
polys. Moreover, the second, redundant call was causing
incorrect viewport shading of previously set custom
normals. Now also setting the ME_AUTOSMOOTH mesh flag
to enable shading with custom normals.

USD importer: vertex normals render black.

Setting point normals by invoking normal_float_to_short_v3(),
instead of directly assigning components as floats.

USD import: Handle empty vertex-varying normals.

If point normals are empty, now invoking BKE_mesh_calc_normals()
to avoid black renders. Since BKE_mesh_calc_normals() requires
edges to be defined, moving point normal calculation to follow
reading polys.

USD import: handle primvars:normals if specified.

Per Pixar UsdGeomPointBased documentation: If 'normals' and
'primvars:normals' are both specified, the latter has precedence.

USD import: orientation bug.

Uninitialized USDMeshReader::m_isLeftHanded flag was
causing the vertex winding orientation to randomly flip,
causing incorrect shading and other issues.

Apply orientation when importing USD normals.

Also added check to ensure normal and loop counts match.

USD Import: Visible Prims Only flag.

Added new option to prune primitives by visibility.
If this option is enabled, invisible prims will
be excluded from the traversal. This only applies
to prims with a non-animating visibility attribute.
Prims with animating visibility will always be imported.

USD import: xform cache constraint parenting bug.

USD_get_transform() was not taking the object's parent
transform into account. Now following the same logic
as in the corresponding Alembic implementation,
ABC_get_transform(), to fix this.

USD import: convert UsdGeomImageables only.

Currently, the importer converts every node in the USD
to a Blender Empty object by default, including nodes
that aren't of UsdGeomImageable type, such as materials
and shaders. This can unnecessarily bloat the Blender
scene with nodes that are not immediately useful. I've
added logic to restrict conversion to nodes that are
UsdGeomImageables, and this can have a dramatic effect
in some cases. For exmaple, the number of nodes imported
in the Attic scene is reduced from 1,025 to 238 with
this change.

USD import: uniform interpolation normals.

Added logic to import USD normals with uniform interpolation
(i.e., per-face normals) as Blender loop normals.

USD Importer: dependency graph update crash.

Fixed occasional random crash due to simultaneous update
to the dependency graph from multiple threads. A typical
collision might happen when the USD import job thread
is creating a material while the window manager in the
main thread is processing event notifiers. The current
fix is to remove the ND_FRAME flag from the call to
WM_jobs_timer(), which prevents ED_update_for_newframe()
from being called from wm_event_do_notifiers().

USD Import: support instance collections.

Added an experimental Instancing option to support
importing USD scenegraph instances as Blender
instanced collections.

Formatting fixes.

USD Import: Minor instancing fixes.

Small edit to the Instancing import option parameter
description. Now allowing setting null instance collection
on USDInstanceReader objects.

USD Import: UV fixes.

Made the following updates to fix UV import errors:
Added logic to read UVs with Vertex interpolation and
indexed UVs. Fixed incorrect UV winding order for
left-handed orientation.

USD Import: USDPrimReader uninitialized members.

Fixed bug where several USDPrimReader class member variables
were not being intialized in the constructor. One of the
previously uninitialized variables stores the reference count,
which was causing memory leaks in some cases.

USD Import: USD Preview conversion.

Added experimental Import USD Preview option to convert
UsdPreviewSurface shaders to Principled BSD shader networks,
for an approximate preview. Added new USDMaterialReader
class, which implements this feature. Also added a
Set Material Blend option to automatically set the material
blend method based on the UsdPreviewSurface shader's opacity
and opacityThreshold inputs. Finally, updated the logic so that
no materials will be imported if the Materials option is
not checked.

USD Import: made USDPrimReader destructor virtual.

USD Import format fixes.

USD Import: Scale option fixes.

Updated the code to apply the global scale option, which
was previously ignored. Fixed application of global scale
in the xform reader matrix calculation to only apply the
scale to root objects. Also fixed bug in setting the
is_constant flag and simplified some of the code.

USD Import: Convert to Z Up option.

Added a Convert to Z Up import option and logic to rotate
imported root objects 90 degrees about the X-axis if this
option is checked and the USD stage up-axis is Y.

USD Import: Formatting fixes.

USD Import: simplify xform matrix computation.

Updated the USDXformReader matrix computation function to use the
standard UsdGeomXformable API for querying the prim's local
transform and to determine whether the matrix is time-varying,
rather than explicitly iterating over the UsdGeomXformOps.

USD Import: fixed compiler warning.

USD Import: Light Intensity Scale option.

Added new float import option to scale intensity of imported
lights.

USD Import: fixed compiler warnings.

USD Import: subdiv option default to off.

The Import Subdiv Option can sometimes behave unexpectedly,
losing all sharp edges. Turning this option off by default
until the behavior can be fine tuned.

USD Import: crash on reading no polys.

Fixed a crash where polys were not being read for new meshes when
the MOD_MESHSEQ_READ_POLY flag isn't set. Crash occurs because
downstream code expects polys to exist. Modified the logic to
always read verts and polys for new meshes, regardless of the
value of the read flag.

USD Import: removed printf call.

USD Import: read attributes null data guard.

Checking for null pointer to avoid crash when custom data
allocation fails.

USD Import: import display colors.

Added support for reading the display color primvars. This
option is off by default in the mesh read flags.

USD Import: removed WM_reportf() trace calls.

USD Import: removed commented out code.

USD Import: use pragma once in headers.

Replaced include guards with pragma once directives,
to conform to the conventions used in the existing USD
export code.

USD Import: removed unneeded include guards.

USD Import: added blender::io::usd namespace.

Added blender::io::usd namespace where it was missing, to
conform to the convention used in the USD exporter code.

USD Import: snake case for function names.

As part of code cleanup, now using snake case consistently
for class member function names, to match the convention
used in the USD exporter code.

USD Import: naming convention fix.

To be consistent with the convention used in the USD exporter,
updated class member variables to consistently use snake case
and trailing underscores instead of the m_ prefix.

USD Import: UsdPrimReader interface cleanup.

Removed empty add_cache_modifier() function and made
create_object() abstract.

USD Import: remove UsdPrimReader stage member.

The UsdPrimReader class doesn't need a stage pointer as a
member because the stage can implicitly be accessed
through the prim itself, as long as the prim is valid.
I removed the UsdPrimReader::stage_ member and updated the
relevant constructors and function calls. Also, updated
the various readers to construct schemas directly from
the contained prim, e.g., pxr::UsdLuxRectLight rect_light(prim_)

USD Import: stage reader validation and cleanup.

Added USDStageReader validation and guard against null
stage in the USDStageReader destructor.

USD Import: remove usd_util files.

Moved the two create_reader() implementations from usd_util.(h|cc)
to static functions in UsdStageReader and removed the usd_util
files from the project.

USD Import: fix reader refcount error.

Currently, USDPrimReader reference counts aren't correctly
maintained because the readers are stored in two separate
arrays, in the USDStageReader instance and also in the
import job data, but the reference counts for the readers
is incremented only once. I changed the logic to store
the readers only in the USDStageReader. Now also deleting
the stage reader instance, fixing a small memory leak. Also,
simplified the syntax for for looping over readers, to make
the code more readable.

USD Import: enforcing const correctness.

USD Import: removed unneeded includes.

USD Import: remove unsupported geometry flags.

For now, removing support for reading velocities and
attributes, because the data allocation for these layers
no longer works in the current version of Blender. We will
revisit implementing these features in the future.

USD Import: removed unneeded code.

USD Import: check bound materials on all subsets.

Changed material import logic to check bound materials
on all subsets, not just on the 'materialBind' subsets,
as this is necessary for geometry currently exported
from Houdini.

USD Import: schema initialization and validation.

Now initializing schemas in the constuctor initialization
lists of reader classes. Checking schemas to validate
readers. Additional, minor cleanup.

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

Cleanup: variable naming.

Fix first part of T86501: Crash during resync process.

Code would end up freeing some of the newly created overrides, which
were assigned to the matching linked ID's `newid` pointer, accessed
again further down the code.

Note that this is not a normal expected situation, and it won't give a
proper resync result anyway, but it might happen in some complicated
corner cases, and also quite often when dealing with older .blend files.

IDRemap: Add option to also remap internal runtime ID pointers.

In some cases (advanced, low-level), we also want to remap pointers like
`ID.newid` or `ID.orig_id`.

Only known case currently is `id_delete`, to avoid leaving potential access to freed memory. See next commit and T86501.

LibQuery: Add an option to process internal runtime ID pointers.

In some cases (advanced, low-level code) we also want to process ID
pointers like `ID.newid` or `ID.orig_id`.

USD Import: schema initialization and validation.

Now initializing schemas in the constuctor initialization
lists of reader classes. Checking schemas to validate
readers. Additional, minor cleanup.

Fix T86455: vertex color baking issue with sculpt vertex colors

Baking to Vertex Colors would always bake to sculpt vertex colors (if
such a layer is present) even if those are not enabled in the
experimental preferences. This would bake without an error but leave the
user without a result to look in the viewport.

Now check if sculpt vertex colors are enabled and only bake to them in
that case.

Maniphest Tasks: T86455

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

USD Import: check bound materials on all subsets.

Changed material import logic to check bound materials
on all subsets, not just on the 'materialBind' subsets,
as this is necessary for geometry currently exported
from Houdini.

Merge branch 'master' into compositor-cryptomatte-workflow

USD Import: removed unneeded code.

USD Import: remove unsupported geometry flags.

For now, removing support for reading velocities and
attributes, because the data allocation for these layers
no longer works in the current version of Blender. We will
revisit implementing these features in the future.

USD Import: removed unneeded includes.