diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-02-27 12:24:11 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-02-27 12:37:59 +0300 |
commit | 10162d68e38514aa8741cf9efd9e0f5deac32085 (patch) | |
tree | d78cfe4761c1fef9ef7af434eed0551158c6ff55 /source/blender/makesdna | |
parent | 4952fb1669fb7d254fc26a527b6dc2a44ac2c496 (diff) |
Constraints: replace 'Set Inverse' operator with an eval-time update
This fixes {T70269}.
Before this commit there was complicated code to try and compute the
correct parent inverse matrix for the 'Child Of' and 'Object Solver'
constraints outside the constraint evaluation. This was done mostly
correctly, but did have some issues. The Set Inverse operator now defers
this computation to be performed during constraint evaluation by just
setting a flag. If the constraint is disabled, and thus tagging it for
update in the depsgraph is not enough to trigger immediate evaluation,
evaluation is forced by temporarily enabling it.
This fix changes the way how the inverse matrix works when some of the
channels of the constraint are disabled. Before this commit, the channel
flags were used to filter both the parent and the inverse matrix. This
meant that it was impossible to make an inverse matrix that would
actually fully neutralize the effect of the constraint. Now only the
parent matrix is filtered, while inverse is applied fully. As a result,
pressing the 'Set Inverse' matrix produces the same transformation as
disabling the constraint. This is also reflected in the changed values
in the 'Child Of' unit test.
This change is not backward compatible, but it should be OK because the
old way was effectively unusable, so it is unlikely anybody relied on
it.
The change in matrix for the Object Solver constraint is due to a
different method of computing it, which caused a slightly different
floating point error that was slightly bigger than allowed by the test,
so I updated the matrix values there as well.
This patch was original written by @angavrilov and subsequently updated
by me.
Differential Revision: https://developer.blender.org/D6091
Diffstat (limited to 'source/blender/makesdna')
-rw-r--r-- | source/blender/makesdna/DNA_constraint_types.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/source/blender/makesdna/DNA_constraint_types.h b/source/blender/makesdna/DNA_constraint_types.h index 6caa0ac5a74..670a6c46187 100644 --- a/source/blender/makesdna/DNA_constraint_types.h +++ b/source/blender/makesdna/DNA_constraint_types.h @@ -1096,6 +1096,8 @@ typedef enum eChildOf_Flags { CHILDOF_SIZEY = (1 << 7), CHILDOF_SIZEZ = (1 << 8), CHILDOF_ALL = 511, + /* Temporary flag used by the Set Inverse operator. */ + CHILDOF_SET_INVERSE = (1 << 9), } eChildOf_Flags; /* Pivot Constraint */ @@ -1147,6 +1149,8 @@ typedef enum eCameraSolver_Flags { /* ObjectSolver Constraint -> flag */ typedef enum eObjectSolver_Flags { OBJECTSOLVER_ACTIVECLIP = (1 << 0), + /* Temporary flag used by the Set Inverse operator. */ + OBJECTSOLVER_SET_INVERSE = (1 << 1), } eObjectSolver_Flags; /* ObjectSolver Constraint -> flag */ |