Blender Git Commits

Avoid Brush user count increase when duplicates the Brush

When you create a new Brush, the user count is `2` (caller and fake user). The problem is when you duplicate a Brush, the user count grows to `3` and this is not correct, because is a new brush.

Reviewed By: mont29

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

fix T69772 Collada importer creates wrong fcurves for skeletal animation

Cycles: Add Random Per Island attribute.

The Random Per Island attribute is a random float associated with each
connected component (island) of the mesh. It is particularly useful
when artists want to add variations to meshes composed of separate
units. Like tree leaves created using particle systems, wood planks
created using array modifiers, or abstract splines created using AN.

Reviewed By: Sergey Sharybin, Jacques Lucke

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

refactor: collada importer: calculate fcurve index only once when creating an fcurve

Cleanup: spelling, clang-format

Cleanup: warnings

Fix T69530: Do Not Refresh Bookmarks While Moving File Browser

Improve laggy performance while moving File Browser by not refreshing fsMenu lists in its init.

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

Reviewed by Bastien Montagne

Cleanup/CMake: Remove dormant windows codesign code

This was added years ago to prepare for code-signing the executable
but was never used, buildbots use a different mechanism now to sign
so no need to keep this around.

Fix preview Template-ID broken in vertical layouts

Old code worked pure "by luck". I fixed an error in rB5bcb0c993503 that
made this "luck" go away. Special handling is needed for the preview
layout.

Gpencil: Fix unreported memory leak duplicating Brush

When the brush is duplicated, the grease pencil settings doesn't need to be recreated.

Allow deletion of directories in the file browser

This diff allows to delete directories in the file browser. The commit https://developer.blender.org/rB8825250f5a85c0c16e74ed144dd2b4a7d752042f did not include this feature.

Reviewed By: Severin

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

Anim: added BKE_object_moves_in_time(object) function

This function exposes the already-existing static `object_moves_in_time()`
function, and optionally recursively checks the parent object for
animatedness as well.

I also added checking `AnimData::overrides` to
`BKE_animdata_id_is_animated()`. This ensures that, apart from the optional
recursion to the parent object, the function has the same functionality.

Anim: made object param of BKE_animdata_id_is_animated() const

No functional changes.

Alembic export: don't assume transform is always animated

Instead of always writing the transform on every frame, it's now checked
whether the object is animated at all. This could be made stricter to
reduce false positives, for example by checking FCurves and drivers to
see whether translation/rotation/scale is animated. However, this
approach is already better than the `return true` we had before.

This commit adds the BKE_animdata_id_is_animated(id) function, which
returns true if the ID datablock has non-empty animation data. This is
determined by checking the the active action's fcurves, the drivers, and
NLA tracks.

Removed `BKE_object_is_animated()`

The function isn't used anywhere, and it's deceptively returning false
negatives. For example, `modifier_dependsOnTime()` will return `false`
for hook modifiers, even when the hook target is animated. Querying the
depsgraph for dependency on the time source would be a better approach.

Made ob param of `modifiers_getVirtualModifierList` const

The `modifiers_getVirtualModifierList()` function previously took a non-
const `Object *ob` parameter, preventing it from being called from more
restrictive functions. Since the function doesn't modify the passed
object, it could easily be made const.

No functional changes.

Clarified comment about `modifiers_getVirtualModifierList()`

The old comment was outdated and incorrect.

No functional changes.

Clip Editor: Remove Image label from selector

Unfortunately, it didn't turn out to be as great as I've hoped to it
will.

The issue is: the label eats too much space, making selector buttons
and image name to be barely readable. Initial idea of making the panel
somewhat wider didn't work either: due to the sizing policy of label
it takes a lot of panel width to make image name readable.

Fix T71924: Baking quaternion keyframes gives discontinuous values

PyAPI: add Quaternion.make_compatible