IK with non uniform scale broke the animation tools #51971

New Issue

Paolo Acampora · 2017-07-04T15:57:47+02:00

Paolo Acampora commented

2017-07-04 15:57:47 +02:00

Blender Version
Broken: 2.78
Worked: 2.77

Non uniform transforms generated by the ik constraint cannot be used by blender operators: Apply visual transform to pose, Animation bake, Copy Attribute, FK/IK snap, none of them will work

Exact steps for others to reproduce the error
Attached blend file contains a very simple ik setup, and a copy with a baked animation, which behaves different. Baking the animation, applying to pose, storing the matrices and setting them, will not end up with the same pose.

This happens since non uniform scale IK has been introduced, and it's fixed disabling USE_NONUNIFORM_SCALE at build time

ik-stretch.blend

**Blender Version** Broken: 2.78 Worked: 2.77 Non uniform transforms generated by the ik constraint cannot be used by blender operators: **Apply visual transform to pose**, **Animation bake**, **Copy Attribute**, **FK/IK snap**, none of them will work **Exact steps for others to reproduce the error** Attached blend file contains a very simple ik setup, and a copy with a baked animation, which behaves different. Baking the animation, applying to pose, storing the matrices and setting them, will not end up with the same pose. This happens since non uniform scale IK has been introduced, and it's fixed disabling USE_NONUNIFORM_SCALE at build time [ik-stretch.blend](https://archive.blender.org/developer/F651793/ik-stretch.blend)

Paolo Acampora commented

2017-07-04 15:57:47 +02:00

Changed status to: 'Open'

Paolo Acampora commented

2017-07-04 15:57:47 +02:00

Added subscriber: @PaoloAcampora

Paolo Acampora commented

2017-07-04 16:01:27 +02:00

Added subscribers: @icappiello, @LucioRossi, @dragostanasie, @ideasman42, @mont29, @BrendonMurphy

Sergey Sharybin commented

2017-07-05 17:21:57 +02:00

Added subscriber: @Sergey

Campbell Barton was assigned by Sergey Sharybin

2017-07-05 17:21:57 +02:00

Sergey Sharybin commented

2017-07-05 17:21:57 +02:00

Not that familiar with this feature, it is implemented by @campbellbarton.

Campbell, mind having a look here?

Not that familiar with this feature, it is implemented by @campbellbarton. Campbell, mind having a look here?

blender-admin commented

2017-07-05 23:15:12 +02:00

This issue was referenced by f05f2f0336

This issue was referenced by f05f2f0336025f00a299ec930151a155d55c6ab3

Brecht Van Lommel commented

2017-07-06 01:34:57 +02:00

Added subscriber: @brecht

Brecht Van Lommel commented

2017-07-06 01:34:57 +02:00

I think D2088 changed two things:

Non-uniform scale is no longer inherited when applying the IK transforms, to match the IK solver normalizing the matrix and making the scale uniform.
IK stretch no longer affects the X and Z axes. I think this was an unintentional change. This is demonstrated in ik-stretch.blend, but not necessarily the reason the scripts and tools are broken.

There's a few different scaling cases:

No scale or uniform scale: no problem.
Non-uniform scale with inherit scale off, as in ik_non_uniform_scale.blend from D2088. This can probably be fixed without breaking the scripts and tools described in this ticket. P513 is an attempt at that but isn't quite right in all cases.
Non-uniform scale with inherit scale on. This is when you get shearing transforms which the IK solver can't represent or solve. Arguably shearing bones are never actually what you want, and perhaps not the important case in practice. This works a bit better after D2088 but can still fail.

I think [D2088](https://archive.blender.org/developer/D2088) changed two things: * Non-uniform scale is no longer inherited when applying the IK transforms, to match the IK solver normalizing the matrix and making the scale uniform. * IK stretch no longer affects the X and Z axes. I think this was an unintentional change. This is demonstrated in `ik-stretch.blend`, but not necessarily the reason the scripts and tools are broken. There's a few different scaling cases: * No scale or uniform scale: no problem. * Non-uniform scale with inherit scale off, as in `ik_non_uniform_scale.blend` from [D2088](https://archive.blender.org/developer/D2088). This can probably be fixed without breaking the scripts and tools described in this ticket. [P513](https://archive.blender.org/developer/P513.txt) is an attempt at that but isn't quite right in all cases. * Non-uniform scale with inherit scale on. This is when you get shearing transforms which the IK solver can't represent or solve. Arguably shearing bones are never actually what you want, and perhaps not the important case in practice. This works a bit better after [D2088](https://archive.blender.org/developer/D2088) but can still fail.

Brendon Murphy commented

2017-07-12 11:24:06 +02:00

hi, we need some resolve here, people are waiting for this to be fixed.
Thanks

hi, we need some resolve here, people are waiting for this to be fixed. Thanks

Paolo Acampora commented

2017-07-12 11:59:55 +02:00

Sorry I could not dig into coding, busy days. The way I see it, if there is interest in keeping this feature but there is no way to make sure it behaves properly, it should definetely be an optional setting. This way the user can willingly activate non uniform scaling, and renounce the benefit of the animation tools in exchange.

In order to do this, we would need pose.ik_param to return a parameters class rather than a None, even when the solver is set to legacy, so some work would be required on the rna and of course on properties_data_armature.py

Regards

Sorry I could not dig into coding, busy days. The way I see it, if there is interest in keeping this feature but there is no way to make sure it behaves properly, it should definetely be an optional setting. This way the user can willingly activate non uniform scaling, and renounce the benefit of the animation tools in exchange. In order to do this, we would need `pose.ik_param` to return a parameters class rather than a None, even when the solver is set to legacy, so some work would be required on the rna and of course on properties_data_armature.py Regards

Brecht Van Lommel commented

2017-07-17 18:55:22 +02:00

Changed status from 'Open' to: 'Resolved'

Brecht Van Lommel closed this issue

2017-07-17 18:55:22 +02:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

IK with non uniform scale broke the animation tools #51971