Thanks for the time spent in investigation and working on the patch. However, I don't think this is a good way to go. There are few aspects:
Think this is a nicer fix in absolute, for sure. But we do need Sergey to review this (am not maintaining this code, only fixed the issue in the less intrusive way possible, because maintainer is away for two weeks and this was pretty serious bug ;) ).
Committed as rBrB7e836bde11ce, thanks.
Get context in diff...
Tweak a comment
I don't think this needs to be reviewed by me unless @Sergey Sharybin (sergey) wants an extra pair of eyes.
SolveCamera operator expect K4 is constant same as K3 coefficient
Thanks for the update!
Wow, that's a lot. Almost makes feel ashamed for not knowing it's such a common model :)
I am curious to know with which software the interoperability is improved with this patch. Maybe it then will be possible to expose this model in a more VFX artist friendly way.
Will classify as bug for now.
Changing the handle type seems to play an important part, animation of just the point positions seems to be fine.
I have this on the list for tomorrow to look at again
@Philipp Oeser (lichtwerk) mask surely shouldn't be mangled after modifying it.
Could be missing DEG_id_update_tag call. How do i reproduce the issue with the attached file?