- Amsterdam, Netherlands
- https://cessen.com
-
Animator, rigger, and software developer. Currently working at the Blender Institute as a developer on Blender's animation system.
Been using Blender since 1998, and worked on Big Buck Bunny and Sintel (two of Blender's open movie projects).
- Joined on
2003-03-21
Indeed, in retrospect neither of these are obvious. I'll add a comment explaining this.
For posterity in this PR: the two things you bring up here are actually related: it's the call to…
Also, I went to go add a case to the unit tests for this method, and discovered... I forgot to make a unit test for it!
Because I'm a goofball! Thanks for catching that.
Action
and AnimData
Also the
layout.template_search()
call for the property selector relies on having a single RNA property that can produce the values it presents.
Oh, so is the primary reason for this…
Happy to land this as-is for now, with the caveat that I would like us to remove last_slot_handle
from RNA before Baklava makes its way into an official release.
Ooooh, that makes sense now. Thanks for the description.
One thing to note: this has quadratic performance due to the call to fcurve_assign_to_channel_group()
in the loop, which itself has linear performance. I think that's fine for now, and we can…