Blender Git Loki
Git Commits -> Revision 33b8007
Revision 33b8007 by Alexander Gavrilov (temp-angavrilov-constraints) June 22, 2021, 13:43 (GMT) |
Copy Transforms: implement Remove Target Shear and more Mix options. This constraint can be naturally viewed as a prototype for a future 4x4 matrix math node (or subset thereof), since its basic semantics already is matrix assignment. Thus it makes sense to add math options to this constraint to increase flexibility in the meantime. This patch adds support for several operations that would be useful: - An option to remove shear in the incoming target matrix. Shear is known to cause issues for various mathematical operations, so an option to remove it at key points is useful. Constraints based on Euler like Copy Rotation and Limit Rotation already have always enabled shear removal built in, because their math doesn't work correctly with shear. In the future node system shear removal would be a separate node (and currently Limit Rotation can be used as a Remove Shear constraint). However removing shear from the result of the target space conversion before mixing (similar to Copy Rotation) has to be built into Copy Transforms itself as an option. - More ways to combine the target and owner matrices. Similar to multiple Inherit Scale modes for parenting, there are multiple ways one may want to combine matrices based on context. This implements 3 variants for each of the Before/After modes (one of them already existing). - Full implements regular matrix multiplication as the most basic option. The downside is the risk of creating shear. - Aligned emulates the 'anti-shear' Aligned Inherit Scale mode, and basically uses Full for location, and Split for rotation/scale. (This choice already existed.) - Split Channels combines location, rotation and scale separately. Looking at D7547 there is demand for Split Channels in some cases, so I think it makes sense to include it in Copy Transforms too, so that the Mix menu items can be identical for it and the Action constraint. Differential Revision: https://developer.blender.org/D9469 |
Commit Details:
Full Hash: 33b80078048747b38f630c7a8f6800fb5bd77a9f
Parent Commit: adff34e
Lines Changed: +119, -11
7 Modified Paths:
/release/scripts/startup/bl_ui/properties_constraint.py (+1, -0) (Diff)
/source/blender/blenkernel/intern/constraint.c (+33, -3) (Diff)
/source/blender/blenlib/BLI_math_matrix.h (+1, -0) (Diff)
/source/blender/blenlib/intern/math_matrix.c (+22, -0) (Diff)
/source/blender/editors/transform/transform_convert.c (+4, -1) (Diff)
/source/blender/makesdna/DNA_constraint_types.h (+16, -1) (Diff)
/source/blender/makesrna/intern/rna_constraint.c (+42, -6) (Diff)
/source/blender/blenkernel/intern/constraint.c (+33, -3) (Diff)
/source/blender/blenlib/BLI_math_matrix.h (+1, -0) (Diff)
/source/blender/blenlib/intern/math_matrix.c (+22, -0) (Diff)
/source/blender/editors/transform/transform_convert.c (+4, -1) (Diff)
/source/blender/makesdna/DNA_constraint_types.h (+16, -1) (Diff)
/source/blender/makesrna/intern/rna_constraint.c (+42, -6) (Diff)