I think it might make sense to add this above:
Hm this is getting very messy.
If we support multi-frame editing? Do we?
Might also be the 6%
spacing which is very low.
Done. Though fix has become quite chaotic due to "vertex_group_remove_from" case. If you prefer I can create a separate PR for that
No that's fine imo.
Much better! Though I still think that it's best to pass in scene->r.cfra
from the operators to the kernel functions rather than the entire scene
.
Since we're not using the Scene
and just the frame, it would be better to pass the frame
into the function.
Since this seems like a pretty annoying regression, will reopen.
Thanks. I think what I missed is that there is GreasePencil::set_active_layer
which does call autolock_inactive_layers
. Makes total sense now that we need to remove this from GreasePencil::set_active_node
.