Copying the flags resolves, but this can be an update problem in dephsgraph.
So it needs a bigger investigation.
Ref T59713
Details
Details
- Reviewers
Joshua Leung (aligorith) Sergey Sharybin (sergey) Brecht Van Lommel (brecht) - Maniphest Tasks
- T59713: Armature layer dots not updated on changes
T67001: Studio and Flat lightning in single color do not respect clipping region. - Commits
- rB709f126e8143: Fix T59713: Armature layer dots not updated on changes
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- master
- Build Status
Buildable 4107 Build 4107: arc lint + arc unit
Event Timeline
Comment Actions
Don't think this is an update problem in dependency graph: the layers act as "input" to it.
The operator seems to be missing a DEG_id_tag_update with ID_RECALC_GEOMETRY for the object where armatures are added to.
P.S. I am confused with title of this revision and the actual changes, not sure why this will be related on clipping region.
Comment Actions
layer and layer_protected are user-edited data. If there is a problem with those when joining armature, that's something to be submitted as a separate fix.
The original report is only about layer_used runtime data, which controls the drawing of dots in the UI. The problem is that this is not getting refreshed properly. This used to happen in the drawing code, but was no longer working for reasons explained in:
rB2b09062defa0: COW Fix: The "layers used" display for armatures did not update after bones…
This could be handled as part of the dependency graph, but I'm not sure that's ideal either since this isn't about evaluated data.
The simple solution would be to rename rna_Armature_layer_used_refresh to BKE_armature_refresh_layer_used and move it to blenkernel. Then call that from all operators that can affect layer_used.
Comment Actions
- Rename rna_Armature_layer_used_refresh to BKE_armature_refresh_layer_used and move it to blenkernel
- Call BKE_armature_refresh_layer_used in the join_armature operator
Comment Actions
Looks good, now it can be added to more operators. From what I can tell these need it:
- ARMATURE_OT_bone_primitive_add
- ARMATURE_OT_delete
- ARMATURE_OT_dissolve
- ARMATURE_OT_fill
- ARMATURE_OT_merge
- ARMATURE_OT_separate
- ARMATURE_OT_bone_layers
- POSE_OT_bone_layers
Comment Actions
- Simplify parameter of the BKE_armature_refresh_layer_used;
- Call BKE_armature_refresh_layer_used in ARMATURE_OT_bone_primitive_add, ARMATURE_OT_delete, ARMATURE_OT_dissolve and ARMATURE_OT_separate
I wonder why it is necessary to refresh the layer_used when calling the operators ARMATURE_OT_fill, ARMATURE_OT_bone_layers and POSE_OT_bone_layers
Although ARMATURE_OT_fill creates new bones they copy the data from the existing bones and don't change the layers used.
The two other operators do not add, destroy or move bones between layers.
Comment Actions
Fill adds the bone on the active layer as far as I can tell, and that one may be empty and become used.
The bone_layers operators move bones to another layer.
Comment Actions
Adding a bone with shift + A also does not seem to update the dots in the latest patch.
Comment Actions
- Call BKE_armature_refresh_layer_used in ARMATURE_OT_fill, ARMATURE_OT_bone_layers and POSE_OT_bone_layers
You're right, ARMATURE_OT_fill adds the bones to the active layers
And I confused the layers parameter of the other operators
Comment Actions
I also noticed adding a new armature with shift + A in object mode does not immediately show used layers, when entering edit mode.
| source/blender/makesrna/intern/rna_armature.c | ||
|---|---|---|
| 303 | This line should be moved into BKE_armature_refresh_layer_used, otherwise layers are never cleared. | |