blender.git
3 months agoMerge branch 'blender-v2.93-release'
Aaron Carlisle [Fri, 28 May 2021 01:27:48 +0000 (21:27 -0400)]
Merge branch 'blender-v2.93-release'

3 months agoDocs: Update RNA to User Manual mappings
Aaron Carlisle [Fri, 28 May 2021 01:25:31 +0000 (21:25 -0400)]
Docs: Update RNA to User Manual mappings

3 months agoFix T86465: Annotation Tool is missing in VSE Preview toolbar
Antonio Vazquez [Thu, 27 May 2021 17:44:53 +0000 (19:44 +0200)]
Fix T86465: Annotation Tool is missing in VSE Preview toolbar

Added missing topbar in VSE.

Also added the Stabilizer options to Topbar for all modes.

Reviewed By: mendio, pepeland

Maniphest Tasks: T86465

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

3 months agoFix own crash in today's rBf68288a8746f.
Bastien Montagne [Thu, 27 May 2021 17:30:23 +0000 (19:30 +0200)]
Fix own crash in today's rBf68288a8746f.

3 months agoLibOverride: refactor recursive resync.
Bastien Montagne [Thu, 27 May 2021 13:54:26 +0000 (15:54 +0200)]
LibOverride: refactor recursive resync.

We need to re-evaluate what needs to be resynced after each step of
processing overrides from a given 'indirect level' of libraries.
Otherwise, recusrive overrides (overrides of linked overrides) won't
work.

Note that this should not change too much in practice currently, since
there are other issues with recursive overrides yet.

Also, checks (CLOG errors) added show that some ID (node trees) seem to
be detected as needing resynced even after beig just resynced, this
needs further investigation still. Could be though that it is due to
limit currently set on nodetrees, those are always complicated
snowflakes to deal with...

3 months agoFix T88614: Mixdown crashes Blender 2.92.0 and 3.0.0 Alpha
Jörg Müller [Thu, 27 May 2021 17:05:17 +0000 (19:05 +0200)]
Fix T88614: Mixdown crashes Blender 2.92.0 and 3.0.0 Alpha

The problem is caused by the most recent ffmpeg version (4.4) which
needs channels to be set when submitting a frame for encoding.

3 months agoFix T88625: Multiobject UV hiding/unhiding does not work with UV_SYNC_SELECTION
Philipp Oeser [Thu, 27 May 2021 13:30:02 +0000 (15:30 +0200)]
Fix T88625: Multiobject UV hiding/unhiding does not work with UV_SYNC_SELECTION

Oversight in {rB470f17f21c06}.

Hiding was only done for the first mesh, then the operator finished (in
case of UV_SYNC_SELECTION).
Now just continue to the next.

Maniphest Tasks: T88625

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

3 months agoMerge branch 'blender-v2.93-release'
Clément Foucault [Thu, 27 May 2021 15:15:39 +0000 (17:15 +0200)]
Merge branch 'blender-v2.93-release'

3 months agoRevert "EEVEE: Ensure Reflection: Use new implementation"
Clément Foucault [Thu, 27 May 2021 15:10:17 +0000 (17:10 +0200)]
Revert "EEVEE: Ensure Reflection: Use new implementation"

Both before and after can have artifacts with some normal maps, but this seems to give
worse artifacts on average which are not worth the minor performance increase.

This reverts commit 5c4d24e1fd752a8a89d44d05e8e3f9b31f2d7db0.

Ref T88368, D10084

3 months agoGeometry Nodes: Expose texture and material inputs to modifier
Hans Goudey [Thu, 27 May 2021 15:06:08 +0000 (11:06 -0400)]
Geometry Nodes: Expose texture and material inputs to modifier

This allows choosing material and texture sockets for the group input
node in the modifier. Note that currently grease pencil materials are
displayed in the list, even though grease pencil data is not supported
yet by geometry nodes. That is more complicated to fix in this case,
since we use IDProperties to store the dynamic exposed inputs.

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

3 months agoCleanup: Fix forward declaring class with "struct"
Hans Goudey [Thu, 27 May 2021 15:00:00 +0000 (11:00 -0400)]
Cleanup: Fix forward declaring class with "struct"

3 months agoFix build error: Make CurveEval a struct
Hans Goudey [Thu, 27 May 2021 14:50:43 +0000 (10:50 -0400)]
Fix build error: Make CurveEval a struct

We need a pointer to this in DNA, which means it cannot be a class.

3 months agoCleanup: rename blender-launcher source file.
Ray Molenkamp [Thu, 27 May 2021 14:10:31 +0000 (08:10 -0600)]
Cleanup: rename blender-launcher source file.

blender-laucher.c was not an ideal name for this file
since it's not directly clear it is windows only.

This change renames it to blender_launcher_win32.c
to be more in line with other win32 specific files
we have.

3 months agoGeometry Nodes: Draw curve data in the viewport
Hans Goudey [Thu, 27 May 2021 14:08:40 +0000 (10:08 -0400)]
Geometry Nodes: Draw curve data in the viewport

This patch adds relatively small changes to the curve draw
cache implementation in order to draw the curve data in the
viewport. The dependency graph iterator is also modified
so that it iterates over the curve geometry component, which
is presented to users as `Curve` data with a pointer to the
`CurveEval`

The idea with the spline data type in geometry nodes is that
curve data itself is only the control points, and any evaluated
data with faces is a mesh. That is mostly expected elsewhere in
Blender anyway. This means it's only necessary to implement
wire edge drawing of `CurveEval` data.

Adding a `CurveEval` pointer to `Curve` is in line with changes
I'd like to make in the future like using `CurveEval` in more places
such as edit mode.

An alternate solution involves converting the curve wire data
to a mesh, however, that requires copying all of the data, and
since avoiding it is rather simple and is in-line with future plans
anyway, I think doing it this way is better.

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

3 months agoFix T88452: Point Separate crash on curve component
Hans Goudey [Thu, 27 May 2021 13:37:19 +0000 (09:37 -0400)]
Fix T88452: Point Separate crash on curve component

The point separate node should create a point cloud from control points
in this case, but for now disable the node on curves to avoid the crash.

3 months agoCleanup: Use consistent variable names
Hans Goudey [Thu, 27 May 2021 13:27:08 +0000 (09:27 -0400)]
Cleanup: Use consistent variable names

3 months agoCleanup: rename BKE_main_id_{clear_newpoins => newptr_and_tag_clear}
Campbell Barton [Thu, 27 May 2021 12:44:02 +0000 (22:44 +1000)]
Cleanup: rename BKE_main_id_{clear_newpoins => newptr_and_tag_clear}

It wasn't obvious this function cleared the tag as well.

3 months agoCleanup: remove duplicate LIB_TAG_NEW untag code
Erik Abrahamsson [Thu, 27 May 2021 12:40:30 +0000 (22:40 +1000)]
Cleanup: remove duplicate LIB_TAG_NEW untag code

This patch removes unnecessary calls to `BKE_main_id_tag_all` where the
same job is done by `BKE_main_id_clear_newpoins` on the following line.

Reviewed By: campbellbarton, mont29

Ref D11379

3 months agoLineArt: List Optimization for tile linked data.
YimingWu [Thu, 27 May 2021 12:33:02 +0000 (20:33 +0800)]
LineArt: List Optimization for tile linked data.

Use array instead of ListBase for line art
bounding area linked triangles and edges.

Reviewed By: Sebastian Parborg (zeddb)

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

3 months agoCleanup: Line art variable naming.
YimingWu [Thu, 27 May 2021 10:32:43 +0000 (18:32 +0800)]
Cleanup: Line art variable naming.

Change `reln` to `eln`.

Reviewed By: Sebastian Parborg (zeddb)

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

3 months agoLibOverride: Add heuristic protection against infinite loop due to libraries inter...
Bastien Montagne [Thu, 27 May 2021 09:53:44 +0000 (11:53 +0200)]
LibOverride: Add heuristic protection against infinite loop due to libraries inter-dependencies.

This is not supposed to happen, but better be safe than sorry, and
assume it is beyond unlikely that someone would use chains of over 10k
linked libraries.

3 months agoCleanup: Unused include in Cycles
Sergey Sharybin [Thu, 27 May 2021 09:35:38 +0000 (11:35 +0200)]
Cleanup: Unused include in Cycles

3 months agoRefactor: Move display pass to Cycles viewport parameters
Sergey Sharybin [Thu, 27 May 2021 09:31:03 +0000 (11:31 +0200)]
Refactor: Move display pass to Cycles viewport parameters

Allows to centralize storage and modification checks in a single place,
avoiding duplication in the synchronization code.

Ideally we would somehow be able to more granularly modify Cycles side
objects. Leaving this for a future decision, because it might be better
to implement it as a graph on the sync side.

3 months agoCleanup: Redundnat member init in Cycles viewport parameters
Sergey Sharybin [Thu, 27 May 2021 09:24:05 +0000 (11:24 +0200)]
Cleanup: Redundnat member init in Cycles viewport parameters

3 months agoRefactor: Naming in Cycles viewport methods
Sergey Sharybin [Thu, 27 May 2021 09:16:40 +0000 (11:16 +0200)]
Refactor: Naming in Cycles viewport methods

Makes it more explicit they operate on shading/light.

Gives room to move more viewport related settings into this class and
cover with specific or generic modification checks.

3 months agoCleanup: Use logical OR in Cycles background shader
Sergey Sharybin [Thu, 27 May 2021 09:11:49 +0000 (11:11 +0200)]
Cleanup: Use logical OR in Cycles background shader

3 months agoCleanup: const qualifier of return type
Sergey Sharybin [Thu, 27 May 2021 09:10:37 +0000 (11:10 +0200)]
Cleanup: const qualifier of return type

3 months agoRefactor: Remove friend class from Cycles viewport parameters
Sergey Sharybin [Thu, 27 May 2021 09:07:58 +0000 (11:07 +0200)]
Refactor: Remove friend class from Cycles viewport parameters

Such pattern should only be used when it is really needed. Otherwise
just stick to a more regular design, without worrying who is the user
of the class. Otherwise it will be annoying to subclass or unit test.

3 months agoRefactor: Rename pass accessor in viewport parameters
Sergey Sharybin [Thu, 27 May 2021 08:42:22 +0000 (10:42 +0200)]
Refactor: Rename pass accessor in viewport parameters

No need to state that it is a viewport display pass, since the method
is within viewport parameters it is implied that parameters do belong
to the viewport.

Brings this code closer to the Cycles-X branch.

3 months agoFix (studio-reported) infinite loop in resync code.
Bastien Montagne [Thu, 27 May 2021 08:47:02 +0000 (10:47 +0200)]
Fix (studio-reported) infinite loop in resync code.

Very stupid mistake in libraries indirect-level building code, was not
skipping 'loop-back' ID pointers.

Note that we also need some level of checks for the case where there
would be an actual dependency loop between libraries, this is not
supposed to be possible, but better be safe than sorry. Will add in next
commit.

3 months agoCleanup: inconsistent parameter name
Jacques Lucke [Thu, 27 May 2021 08:46:43 +0000 (10:46 +0200)]
Cleanup: inconsistent parameter name

3 months agoCleanup: inconsistent parameter name
Jacques Lucke [Thu, 27 May 2021 08:41:40 +0000 (10:41 +0200)]
Cleanup: inconsistent parameter name

3 months agoCleanup: simplify logic for copying vector button as text
Campbell Barton [Thu, 27 May 2021 07:44:30 +0000 (17:44 +1000)]
Cleanup: simplify logic for copying vector button as text

An arbitrary size offsets was used in float_array_to_string,
simplify the loop, use exact size limits.

Also rename variables so it's clear which array the length apply to.

3 months agoFix incorrect BLI_snprintf usage
Campbell Barton [Thu, 27 May 2021 07:16:08 +0000 (17:16 +1000)]
Fix incorrect BLI_snprintf usage

Event though in practice this wasn't causing problems as the fixed size
buffers are generally large enough not to truncate text.

Using the result from `snprint` or `BLI_snprintf` to step over a fixed
size buffer allows for buffer overruns as the returned value is the size
needed to copy the entire string, not the number of bytes copied.

Building strings using this convention with multiple calls:

    ofs += BLI_snprintf(str + ofs, str_len_max - ofs);

.. caused the size argument to become negative,
wrapping it to a large value when cast to the unsigned argument.

3 months agoCleanup: specify array sizes, remove warnings in comments
Campbell Barton [Thu, 27 May 2021 07:12:38 +0000 (17:12 +1000)]
Cleanup: specify array sizes, remove warnings in comments

3 months agoCleanup: spelling
Campbell Barton [Thu, 27 May 2021 07:02:44 +0000 (17:02 +1000)]
Cleanup: spelling

3 months agoNodes: fix material node copied over when socket is copied
Jacques Lucke [Thu, 27 May 2021 07:58:45 +0000 (09:58 +0200)]
Nodes: fix material node copied over when socket is copied

This was missing from rB207472930834a2916cf18bbdff51bcd77c6dd0c0.

3 months agoGeometry Nodes: disable multi-threading in evaluator for now
Jacques Lucke [Thu, 27 May 2021 07:43:11 +0000 (09:43 +0200)]
Geometry Nodes: disable multi-threading in evaluator for now

A deadlock could happen under certain circumstances when
geometry nodes is used on multiple objects.
Once T88598 is resolved, multi-threading can be enabled again.

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

3 months agoCleanup: use UndoMesh as links instead of allocating LinkData
Campbell Barton [Thu, 27 May 2021 06:34:41 +0000 (16:34 +1000)]
Cleanup: use UndoMesh as links instead of allocating LinkData

While the advantage isn't large,
it's simpler to skip the intermediate link.

Also remove unused next and previous struct members
from MeshUndoStep_Elem.

3 months agoUndo: resolve inefficient edit-mesh memory use with multiple objects
Campbell Barton [Thu, 27 May 2021 06:14:51 +0000 (16:14 +1000)]
Undo: resolve inefficient edit-mesh memory use with multiple objects

When editing more than 1 object at a time, complete copies of each mesh
were being stored. Now the most recent undo-data for each mesh is used
(when available).

3 months agoCleanup: Remove completed "TODO" comment
Hans Goudey [Thu, 27 May 2021 02:32:53 +0000 (22:32 -0400)]
Cleanup: Remove completed "TODO" comment

Since rBb67fe05d4bea, the dependency graph supports relations
on collection geometry, and the nodes modifier uses that.

3 months agoCleanup: Simplify spline point attribute materialize functions
Hans Goudey [Thu, 27 May 2021 02:22:09 +0000 (22:22 -0400)]
Cleanup: Simplify spline point attribute materialize functions

- Iterate over the mask directly instead of using an index.
- Use Span slice and copy_from instead of a lower level function.

3 months agoGeometry Nodes: Support interpolation between curve domains
Hans Goudey [Thu, 27 May 2021 02:14:59 +0000 (22:14 -0400)]
Geometry Nodes: Support interpolation between curve domains

This commit adds interpolation from the point domain to the spline
domain and the other way around. Before this, spline domain attributes
were basically useless, but now they are quite helpful as a way to use
a shared value in a contiguous group of points.

I implementented a special virtual array for the spline to points
conversion, so that conversion should be close to the ideal performance
level, but there are a few ways we could optimize the point to spline
conversion in the future:
 - Use a function virtual array to mix the point values for each spline
   on demand.
 - Implement a special case for when the input virtual array is one of
   the virtual arrays from the spline point attributes. In other words,
   decrease curve attribute access overhead.

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

3 months agoWin: Add launcher to hide the console window flash
Ray Molenkamp [Thu, 27 May 2021 02:02:35 +0000 (20:02 -0600)]
Win: Add launcher to hide the console window flash

This patch fixes a long-standing complaint from users:
the console window shortly flashing when they start
blender.

This is done by adding a new executable called
blender-launcher.exe which starts blender.exe while
hiding the console.

Any command line parameters given to blender-launcher
will be passed on to blender.exe so it'll be a drop
in replacement.

Starting blender.exe on its own will still function as
a proper console app so no changes required here for
users that use blender for batch processing.

Notable changes:

Registering blender (-R switch) will now register
blender-launcher as the preferred executable.

This patch updates the installer and updates the
shortcuts to start blender-launcher.exe rather
than blender.exe

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

Reviewed by: brecht, harley

3 months agoCleanup: Specify amount of buffers through preprocessor directives
Germano Cavalcante [Thu, 27 May 2021 00:28:05 +0000 (21:28 -0300)]
Cleanup: Specify amount of buffers through preprocessor directives

3 months agoFix T88603: Crash with spline attributes after curve resample
Hans Goudey [Thu, 27 May 2021 01:12:38 +0000 (21:12 -0400)]
Fix T88603: Crash with spline attributes after curve resample

The output curve's spline attribute domain custom data needs to be
reallocated with the correct length after adding the splines.

3 months agoRevert "DrawManager: Use Compute Shader to Update Hair."
Jeroen Bakker [Wed, 26 May 2021 18:32:05 +0000 (20:32 +0200)]
Revert "DrawManager: Use Compute Shader to Update Hair."

This reverts commit 8f9599d17e80254928d2d72081a4c7e0dee64038.

Mac seems to have an error with this change.
```
                 ERROR: /Users/blender/git/blender-vdev/blender.git/source/blender/draw/intern/draw_hair.c:115:44: error: use of undeclared identifier 'shader_src'
                 ERROR: /Users/blender/git/blender-vdev/blender.git/source/blender/draw/intern/draw_hair.c:123:13: error: use of undeclared identifier 'shader_src'
                 ERROR: make[2]: *** [source/blender/draw/CMakeFiles/bf_draw.dir/intern/draw_hair.c.o] Error 1
                 ERROR: make[1]: *** [source/blender/draw/CMakeFiles/bf_draw.dir/all] Error 2
                 ERROR: make: *** [all] Error 2

```

3 months agoCleanup: array-parameter warning with GCC 11
Campbell Barton [Wed, 26 May 2021 16:23:19 +0000 (02:23 +1000)]
Cleanup: array-parameter warning with GCC 11

Pass the string size as this is less error prone in general.

3 months agoRevert "Cycles: optimize ensure_valid_reflection(), reduces render time by about 1%"
Brecht Van Lommel [Wed, 26 May 2021 16:22:43 +0000 (18:22 +0200)]
Revert "Cycles: optimize ensure_valid_reflection(), reduces render time by about 1%"

Both before and after can have artifacts with some normal maps, but this seems to give
worse artifacts on average which are not worth the minor performance increase.

This reverts commit 21bc1a99baa765d81c3203fd2e451681b8a7fd55.

Ref T88368, D10084

3 months agoFix buildbot CUDA/OptiX warnings on macOS
Brecht Van Lommel [Wed, 26 May 2021 15:11:45 +0000 (17:11 +0200)]
Fix buildbot CUDA/OptiX warnings on macOS

Explicitly disable these, rather than relying on them not being found.
Also, don't duplicates the architectures list.

3 months agoGPencil: Cleanup - Conform with RNA naming scheme
Falk David [Wed, 26 May 2021 16:23:25 +0000 (18:23 +0200)]
GPencil: Cleanup - Conform with RNA naming scheme

The newly added `disable_masks_viewlayer` RNA property did not conform
with the RNA naming scheme. This renames it to `use_viewlayer_masks`.

3 months agoFix T88111: Skin modifier assets within invalid face normals
Campbell Barton [Wed, 26 May 2021 06:06:02 +0000 (16:06 +1000)]
Fix T88111: Skin modifier assets within invalid face normals

The skin modifier was moving vertices without updating normals for the
connected faces, this happened when smoothing and welding vertices.

Reviewed By: mont29

Ref D11397

3 months agoCleanup: shadow warning
Campbell Barton [Wed, 26 May 2021 15:30:52 +0000 (01:30 +1000)]
Cleanup: shadow warning

Move reproject_type into an extern, to avoid declaring multiple times.

3 months agoLibOverride: add recursive resync.
Bastien Montagne [Wed, 26 May 2021 14:53:59 +0000 (16:53 +0200)]
LibOverride: add recursive resync.

Recursive resync means also resyncing overrides that are linked from
other library files into current working file.

Note that this allows to get 'working' files even when their
dependencies are out of sync. However, since linked data is never
written/saved, this has to be re-done every time the working file is
loaded, until said dependencies are updated properly.

NOTE: This is still missing the 'report' side of things, which is part
of a larger task to enhance reports regarding both linking, and
liboverrides (see T88393).

----------

Technical notes:

Implementing this proved to be slightly more challenging than expected,
mainly because one of the key aspects of the feature was never done in
Blender before: manipulating, re-creating linked data.

This ended up moving the whole resync code to use temp IDs out of bmain,
which is better in the long run anyway (and more aligned with what we
generally want to do when manipulating temp ID data). It should also
give a marginal improvement in performances for regular resync.

This commit also had to carefully 'sort' libraries by level of indirect
usage, as we want to resync first the libraries that are the least directly
used, i.e. libraries that are most used by other libraries.

3 months agoLibOverride: add helper to retrieve override data from an ID.
Bastien Montagne [Wed, 26 May 2021 13:44:51 +0000 (15:44 +0200)]
LibOverride: add helper to retrieve override data from an ID.

Embedded IDs do not own their own override data, but rather use the one
from their owner.

3 months agoIDManagement: Shapekey: add a `owner_get` callback.
Bastien Montagne [Wed, 26 May 2021 13:43:12 +0000 (15:43 +0200)]
IDManagement: Shapekey: add a `owner_get` callback.

Even though shepkeys are not strictly speaking an embedded data, they
share quiet a few points with those, and from liboverride perspective
they are embedded, so...

3 months agoLibOverride: Do not try to generate override data of linked data.
Bastien Montagne [Wed, 26 May 2021 10:43:34 +0000 (12:43 +0200)]
LibOverride: Do not try to generate override data of linked data.

This is obviously not saved, and should never be editable, so was only a
waste of time.

3 months agoID management: remapping: add flag to enforce refcounting handling.
Bastien Montagne [Wed, 26 May 2021 09:52:24 +0000 (11:52 +0200)]
ID management: remapping: add flag to enforce refcounting handling.

While indeally we should only skip refcounting when relevant tag is set,
doing this in remapping code is too risky for now.

Related to previous commit and T88555.

3 months agoID management: Do not assume that `NO_MAIN` means `NO_USER_REFCOUNT`
Bastien Montagne [Wed, 26 May 2021 09:45:27 +0000 (11:45 +0200)]
ID management: Do not assume that `NO_MAIN` means `NO_USER_REFCOUNT`

While this is still very fuzzy in current code, this old behavior makes
it close to impossible to efficiently use out-of-main temp data, as it
implies that we'd need to update refcounts everytime we add something
back into BMain (an 'un-refcount' ID usages when removing from BMain).

Now that we have two separate flags/tags for those two different things,
let's not merge them anymore.

Note that this is somewhat on-going process, still needs more checks and
cleanup. Related to T88555.

3 months agoDrawManager: Use Compute Shader to Update Hair.
Jeroen Bakker [Wed, 26 May 2021 15:02:32 +0000 (17:02 +0200)]
DrawManager: Use Compute Shader to Update Hair.

This patch will use compute shaders to create the VBO for hair.
The previous implementation uses tranform feedback.

Timings master (transform feedback with GPU_USAGE_STATIC between 0.000069s and 0.000362s
Timings transform feedback with GPU_USAGE_DEVICE_ONLY. between 0.000057s and 0.000122s
Timings compute shader between 0.000032 and 0.000092s

Future improvements:
* Generate hair Index buffer using compute shaders: currently done single threaded on CPU, easy to add as compute shader.

Reviewed By: fclem

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

3 months agoGPU: Compute Pipeline.
Jeroen Bakker [Wed, 26 May 2021 14:49:17 +0000 (16:49 +0200)]
GPU: Compute Pipeline.

With the compute pipeline calculation can be offloaded to the GPU.
This patch only adds the framework for compute. So no changes for users at
this moment.

NOTE: As this is an OpenGL4.3 feature it must always have a fallback.

Use `GPU_compute_shader_support` to check if compute pipeline can be used.
Check `gpu_shader_compute*` test cases for usage.

This patch also adds support for shader storage buffer objects and device only
vertex/index buffers.

An alternative that had been discussed was adding this to the `GPUBatch`, this
was eventually not chosen as it would lead to more code when used as part of a
shading group. The idea is that we add an `eDRWCommandType` in the near
future.

Reviewed By: fclem

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

3 months agoGPencil: Add option to disable masks in view layer
Falk David [Wed, 26 May 2021 14:46:00 +0000 (16:46 +0200)]
GPencil: Add option to disable masks in view layer

This patch adds an option in the Layers > Relations panel called "Disable Masks in Render".
When checked, no masks on this layer are included in the render.

Example:
| {F10087680} | {F10087681} |

See T88202 for why this is needed.

Reviewed By: antoniov

Maniphest Tasks: T88202

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

3 months agoMerge branch 'blender-v2.93-release'
Falk David [Wed, 26 May 2021 14:34:03 +0000 (16:34 +0200)]
Merge branch 'blender-v2.93-release'

3 months agoFix: GPencil mask shows in view layer render
Falk David [Wed, 26 May 2021 14:07:03 +0000 (16:07 +0200)]
Fix: GPencil mask shows in view layer render

Currently when rendering the view layer of a grease pencil layer that has
a mask layer attached, the mask layer would show in the rendered image.
This is inconsistent with the default behaviour with no mask on the
grease pencil layer, because it would only render what's on that
particular layer and not anything from any other layer.

This patch makes the masks invisible in the render.

Note: This might seem like not the best solution, but because masks are
just regular grease pencil layers, it's tricky to pass this edge-case to the
drawing code. The way it is handled right now is the best I could come
up with, without making changes that could affect something else.

Reviewed By: antoniov

Maniphest Tasks: T88202

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

3 months agoMerge branch 'blender-v2.93-release'
Jacques Lucke [Wed, 26 May 2021 14:20:24 +0000 (16:20 +0200)]
Merge branch 'blender-v2.93-release'

3 months agoFix T88250: crash when instancing object in disabled collection
Jacques Lucke [Wed, 26 May 2021 14:06:01 +0000 (16:06 +0200)]
Fix T88250: crash when instancing object in disabled collection

This issue was that `BKE_object_eval_uber_data` was not called for
the text object, because its geometry was not dependent upon
and its `is_directly_visible` tag was `false`. The crash happens in
rendering code, because the evaluated data is missing.

This not only affects text objects, but all object types that have a
geometry component that geometry nodes does not support yet.

The solution is to just add the missing dependencies.

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

3 months agoGPencil: Bake GPencil object transforms into a new GPencil object
Antonio Vazquez [Wed, 26 May 2021 13:41:38 +0000 (15:41 +0200)]
GPencil: Bake GPencil object transforms into a new GPencil object

This operator is a common request of animators to convert the transformation (inluding modifiers) of one grease pencil object, into a new object, generating strokes.

Reviewed By: pepeland

Maniphest Tasks: T87424

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

3 months agoNodes: fix threading issues with node ui storage
Jacques Lucke [Wed, 26 May 2021 12:19:01 +0000 (14:19 +0200)]
Nodes: fix threading issues with node ui storage

Calling BKE_nodetree_attribute_hint_add from multiple threads still
was not safe before..
One issue was that context_map embedded its values directly. So
when context_map grows, all NodeUIStorage would move as well.
I could patch around that by using std::unique_ptr in a few places,
but that just becomes too complex for now.
Instead I simplified the locking a bit by adding just locking a mutex
in NodeTreeUIStorage all the time while an attribute hint is added.

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

3 months agoFix T88588: crash when muting node with multi input socket
Jacques Lucke [Wed, 26 May 2021 10:24:56 +0000 (12:24 +0200)]
Fix T88588: crash when muting node with multi input socket

The bug existed before the new evaluator already, but the new evaluator
is more sensitive to this kind of error.

3 months agoCycles: optimize attributes device updates
Kévin Dietrich [Tue, 18 May 2021 11:09:29 +0000 (13:09 +0200)]
Cycles: optimize attributes device updates

When an `AttributeSet` is tagged as modified, which happens after the addition or
removal of an `Attribute` from the set, during the following GeometryManager device
update, we update and repack the kernel data for all attribute types. However, if we
only add or remove a `float` attribute, `float2` or `float3` attributes should not
be repacked for efficiency.

This patch adds some mechanisms to detect which attribute types are modified from
the AttributeSet.

Firstly, this adds an `AttrKernelDataType` to map the data type of the Attribute to
the one used in the kernel as there is no one to one match between the two since e.g.
`Transform` or `float4` data are stored as `float3s` in the kernel.

Then, this replaces the `AttributeSet.modified` boolean with a set of flags to detect
which types have been modified. There is no specific flag type (e.g.
`enum ModifiedType`), rather the flags used derive simply from the
`AttrKernelDataType` enumeration, to keep things synchronized.

The logic to remove an `Attribute` from the `AttributeSet` and tag the latter as modified
is centralized in a new `AttributeSet.remove` method taking an iterator as input.

Lastly, as some attributes like standard normals are not stored in the various
kernel attribute arrays (`DeviceScene::attribute_*`), the modified flags are only
set if the associated standard corresponds to an attribute which will be stored
in the kernel's attribute arrays. This makes it so adding or removing such attributes
does not trigger an unnecessary update of other type-related attributes.

Reviewed By: brecht

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

3 months agoFix particlesystem not duplicating their pointcache in NO_MAIN case.
Bastien Montagne [Tue, 25 May 2021 17:10:34 +0000 (19:10 +0200)]
Fix particlesystem not duplicating their pointcache in NO_MAIN case.

Sharing data between duplicated IDs should be restricted to depsgraph
(CoW) cases, not all NO_MAIN ones...

While this was probably not an issue currently, we aim at using more and
more out-of-main IDs for temp data processing.

NOTE: Somewhat related to T88555, and similar issue as the one fixed in
rBdfb963c70df5.

3 months agoFix T88552: Cycles changing Render Passes in viewport does not work
Sergey Sharybin [Wed, 26 May 2021 09:16:47 +0000 (11:16 +0200)]
Fix T88552: Cycles changing Render Passes in viewport does not work

3 months agoCleanup: Simplify Cycles viewport parameters
Sergey Sharybin [Wed, 26 May 2021 09:07:05 +0000 (11:07 +0200)]
Cleanup: Simplify Cycles viewport parameters

Use early output and access shading RNA object only once.

3 months agoCompositor: Use BLI_color in convert alpha node.
Jeroen Bakker [Wed, 26 May 2021 07:28:01 +0000 (09:28 +0200)]
Compositor: Use BLI_color in convert alpha node.

Recently the CPP colors module landed in master. This patch will use the
new module in the convert alpha node.

3 months agoCleanup: Remove unused argument in Cycles sync
Sergey Sharybin [Wed, 26 May 2021 08:45:27 +0000 (10:45 +0200)]
Cleanup: Remove unused argument in Cycles sync

Makes it easier to see where exactly the viewport is used.

3 months agoFix T88534: Unable to add a Geometry Node Tree on Volume object
Philipp Oeser [Tue, 25 May 2021 10:02:57 +0000 (12:02 +0200)]
Fix T88534: Unable to add a Geometry Node Tree on Volume object

Volumes are supported, poll corrected.

Maniphest Tasks: T88534

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

3 months agoFix T88531: Mantaflow problem with geometry nodes
Philipp Oeser [Tue, 25 May 2021 14:34:10 +0000 (16:34 +0200)]
Fix T88531: Mantaflow problem with geometry nodes

Objects modified by geometry nodes modifiers were not caught as being
"dynamic".

Now add this modifier type to the list of modifiers making them "dynamic"
in the eyes of mantaflow.

(noticed by @sebbas in chat)

Maniphest Tasks: T88531

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

3 months agoFix T88566: Mantaflow inflow with shapekeys is not working anymore
Philipp Oeser [Tue, 25 May 2021 14:27:34 +0000 (16:27 +0200)]
Fix T88566: Mantaflow inflow with shapekeys is not working anymore
(regression)

Code was actually checking for shapekeys, but these were not detected
properly (some effects like shape keys are added as virtual modifiers
before the user created modifiers)

Now go over virtual modifiers as well.

Maniphest Tasks: T88566

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

3 months agoMerge branch 'blender-v2.93-release'
Philipp Oeser [Wed, 26 May 2021 07:55:47 +0000 (09:55 +0200)]
Merge branch 'blender-v2.93-release'

3 months agoFix T88251: "Operator Cheat Sheet" Crash
Philipp Oeser [Tue, 25 May 2021 15:46:17 +0000 (17:46 +0200)]
Fix T88251: "Operator Cheat Sheet" Crash

Caused by {rB919558854d62}.

Same fix as in {rBdc8a43c8755a} -- let RNA enum item callbacks check
for NULL context.

The NULL context is used to extract items for document generation.

Maniphest Tasks: T88251

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

3 months agoCleanup: Fix short comparison with bool warning
Hans Goudey [Tue, 25 May 2021 18:37:58 +0000 (14:37 -0400)]
Cleanup: Fix short comparison with bool warning

For some reason the hide status is stored in a short and a char
(we cannot have bools in DNA).

3 months agoBump FFmpeg version from 4.2.3 to 4.4
Sybren A. Stüvel [Tue, 25 May 2021 16:58:28 +0000 (18:58 +0200)]
Bump FFmpeg version from 4.2.3 to 4.4

Bump FFmpeg version to 4.4 to fix a problem where it would write the
wrong frame rate. Their old API was deprecated and Blender moved to the
new one in rB8d6264ea12bfac0912c7249f00af2ac8e3409ed1. The new one
produced files with the wrong frame rate, which was fixed in FFmpeg 4.4.

Manifest Task: T88568

Reviewed By: LazyDodo, zeddb

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

3 months agodeps: Fix broken boost link
Ray Molenkamp [Tue, 25 May 2021 16:27:48 +0000 (10:27 -0600)]
deps: Fix broken boost link

3 months agoGPencil: Add a use_light option when creating object.
YimingWu [Tue, 25 May 2021 15:45:47 +0000 (23:45 +0800)]
GPencil: Add a use_light option when creating object.

This option is default off when creating line art objects
because line art seldom use lighting and the normal data
would be all over the place anyway.

Reviewed By: Antonio Vazquez (antoniov)

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

3 months agoCleanup: Use ListBase in various places in line art.
YimingWu [Tue, 25 May 2021 15:32:04 +0000 (23:32 +0800)]
Cleanup: Use ListBase in various places in line art.

This clarifies the data structures for storing edges
for different calculation stages.

Reviewed By: Sebastian Parborg (zeddb)

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

3 months agoBlenlib: Explicit Colors.
Jeroen Bakker [Tue, 25 May 2021 15:16:35 +0000 (17:16 +0200)]
Blenlib: Explicit Colors.

Colors are often thought of as being 4 values that make up that can make any color.
But that is of course too limited. In C we didn’t spend time to annotate what we meant
when using colors.

Recently `BLI_color.hh` was made to facilitate color structures in CPP. CPP has possibilities to
enforce annotating structures during compilation and can adds conversions between them using
function overloading and explicit constructors.

The storage structs can hold 4 channels (r, g, b and a).

Usage:

Convert a theme byte color to a linearrgb premultiplied.
```
ColorTheme4b theme_color;
ColorSceneLinear4f<eAlpha::Premultiplied> linearrgb_color =
    BLI_color_convert_to_scene_linear(theme_color).premultiply_alpha();
```

The API is structured to make most use of inlining. Most notable are space
conversions done via `BLI_color_convert_to*` functions.

- Conversions between spaces (theme <=> scene linear) should always be done by
  invoking the `BLI_color_convert_to*` methods.
- Encoding colors (compressing to store colors inside a less precision storage)
  should be done by invoking the `encode` and `decode` methods.
- Changing alpha association should be done by invoking `premultiply_alpha` or
  `unpremultiply_alpha` methods.

# Encoding.

Color encoding is used to store colors with less precision as in using `uint8_t` in
stead of `float`. This encoding is supported for `eSpace::SceneLinear`.
To make this clear to the developer the `eSpace::SceneLinearByteEncoded`
space is added.

# Precision

Colors can be stored using `uint8_t` or `float` colors. The conversion
between the two precisions are available as methods. (`to_4b` and
`to_4f`).

# Alpha conversion

Alpha conversion is only supported in SceneLinear space.

Extending:
- This file can be extended with `ColorHex/Hsl/Hsv` for different representations
  of rgb based colors. `ColorHsl4f<eSpace::SceneLinear, eAlpha::Premultiplied>`
- Add non RGB spaces/storages ColorXyz.

Reviewed By: JacquesLucke, brecht

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

3 months agoRevert "Blenlib: Explicit Colors."
Jeroen Bakker [Tue, 25 May 2021 15:03:54 +0000 (17:03 +0200)]
Revert "Blenlib: Explicit Colors."

This reverts commit fd94e033446c72fb92048a9864c1d539fccde59a.
does not compile against latest master.

3 months agoBlenlib: Explicit Colors.
Jeroen Bakker [Tue, 25 May 2021 15:00:14 +0000 (17:00 +0200)]
Blenlib: Explicit Colors.

Colors are often thought of as being 4 values that make up that can make any color.
But that is of course too limited. In C we didn’t spend time to annotate what we meant
when using colors.

Recently `BLI_color.hh` was made to facilitate color structures in CPP. CPP has possibilities to
enforce annotating structures during compilation and can adds conversions between them using
function overloading and explicit constructors.

The storage structs can hold 4 channels (r, g, b and a).

Usage:

Convert a theme byte color to a linearrgb premultiplied.
```
ColorTheme4b theme_color;
ColorSceneLinear4f<eAlpha::Premultiplied> linearrgb_color =
    BLI_color_convert_to_scene_linear(theme_color).premultiply_alpha();
```

The API is structured to make most use of inlining. Most notable are space
conversions done via `BLI_color_convert_to*` functions.

- Conversions between spaces (theme <=> scene linear) should always be done by
  invoking the `BLI_color_convert_to*` methods.
- Encoding colors (compressing to store colors inside a less precision storage)
  should be done by invoking the `encode` and `decode` methods.
- Changing alpha association should be done by invoking `premultiply_alpha` or
  `unpremultiply_alpha` methods.

# Encoding.

Color encoding is used to store colors with less precision as in using `uint8_t` in
stead of `float`. This encoding is supported for `eSpace::SceneLinear`.
To make this clear to the developer the `eSpace::SceneLinearByteEncoded`
space is added.

# Precision

Colors can be stored using `uint8_t` or `float` colors. The conversion
between the two precisions are available as methods. (`to_4b` and
`to_4f`).

# Alpha conversion

Alpha conversion is only supported in SceneLinear space.

Extending:
- This file can be extended with `ColorHex/Hsl/Hsv` for different representations
  of rgb based colors. `ColorHsl4f<eSpace::SceneLinear, eAlpha::Premultiplied>`
- Add non RGB spaces/storages ColorXyz.

Reviewed By: JacquesLucke, brecht

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

3 months agoFix T88096: Baking with OptiX and displacement fails
Patrick Mours [Tue, 25 May 2021 14:56:16 +0000 (16:56 +0200)]
Fix T88096: Baking with OptiX and displacement fails

Using displacement runs the shader eval kernel, but since OptiX modules are not loaded when
baking is active, those were not available and therefore failed to launch. This fixes that by falling
back to the CUDA kernels.

3 months agoGeometry Nodes: Add Shader Curve Nodes
Charlie Jolly [Mon, 24 May 2021 11:14:06 +0000 (12:14 +0100)]
Geometry Nodes: Add Shader Curve Nodes

Convert curve vec and curve rgb shader nodes to geometry nodes, based on node_shader_valToRgb.cc implementation.

3 months agoCleanup: Line art naming changes.
YimingWu [Tue, 25 May 2021 14:11:06 +0000 (22:11 +0800)]
Cleanup: Line art naming changes.

Make variable naming consistent with struct names.

Reviewed By: Sebastian Parborg (zeddb)

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

3 months agoCleanup: Convert to static type directly from CPPType
Hans Goudey [Tue, 25 May 2021 13:52:08 +0000 (09:52 -0400)]
Cleanup: Convert to static type directly from CPPType

3 months agoFix T86956: VSE shading mode ignores Grease Pencil Vertex colors.
Jeroen Bakker [Tue, 25 May 2021 13:10:46 +0000 (15:10 +0200)]
Fix T86956: VSE shading mode ignores Grease Pencil Vertex colors.

Issue is that due to the strange definition of render in grease pencil
(meaning should be rendered similar to rendering). This included normal
viewport rendering in OB_RENDER and OpenGL render in OB_RENDER.

For other rendering modes the overlay vertex opacity would be used. This
patch sets this value to 1 when rendering via a scene strip override.

NOTE: that this isn't a good solution as I expect that users want to use
the opacity of the Grease pencil object. Perhaps the GPencil team has a
better solution for it.

3 months agoUnreported fix: vertex colors overlay not set for new 3d views.
Jeroen Bakker [Tue, 25 May 2021 13:09:38 +0000 (15:09 +0200)]
Unreported fix: vertex colors overlay not set for new 3d views.

Found during researching {T86956}.

3 months agoMerge branch 'blender-v2.93-release'
Richard Antalik [Tue, 25 May 2021 13:09:54 +0000 (15:09 +0200)]
Merge branch 'blender-v2.93-release'

3 months agoFix T88551: Crash accessing uninitialized tool settings
Richard Antalik [Tue, 25 May 2021 13:02:31 +0000 (15:02 +0200)]
Fix T88551: Crash accessing uninitialized tool settings

Tool settings for sequencer were not initialized, which caused crash
when adding strips.

There was fix for same issue in versioning rB0f81dafe6cec, but
subversion was not bumped, so files with uninitialized tool settings
may still exist.

Add `SEQ_tool_settings_get()` accessor function that will initialize
tool settings if they are missing. Change operator code to use
`SEQ_tool_settings_fit_method_get()` function instead of accessing
tool settings directly

Reviewed By: campbellbarton

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

3 months agoCleanup: use nullptr
Jacques Lucke [Tue, 25 May 2021 11:37:50 +0000 (13:37 +0200)]
Cleanup: use nullptr

3 months agoCleanup: remove unused function
Jacques Lucke [Tue, 25 May 2021 11:12:54 +0000 (13:12 +0200)]
Cleanup: remove unused function

3 months agoCleanup: remove unnecessary lambda capture
Jacques Lucke [Tue, 25 May 2021 11:10:30 +0000 (13:10 +0200)]
Cleanup: remove unnecessary lambda capture

3 months agoFix T88549: ID sorting tests.
Bastien Montagne [Tue, 25 May 2021 10:14:43 +0000 (12:14 +0200)]
Fix T88549: ID sorting tests.

Forgot to initialize the ID types array...

Weird though that this only failed on Windows!

Thanks a lot to @deadpin for helping investigating this.