Eyes FACS Units bug in Dev Version

[Miki3d]

Hi there!
I’m using Genesis 8.1 Female and FACS Units with the Diffeomorphic Dev Version, since it imports the “Eye Look Auto” unit (doesn’t show up in the official version).
Anyway I got a bug.
When importing a pose from Daz, Eyes Blink and Eyes Squint values are not imported.
I noticed that they have no space between names, but I don’t know if this matters or not.
Thanks

[Alessandro Padovani]

I can confirm the issue. There’s also something odd in many facs units since they become boolean. That is, after importing the test pose facs.duf, they become either zero or one in range. For example eye blink left and right. I use the DAZ limits in the global settings.

Test pose included facs.duf.

steps:

1. import G8.1F and merge rigs
2. import facs units
3. import the test pose facs.duf, most sliders become boolean and some unit values are not imported

‌

‌

‌

[Alessandro Padovani]

G8.1F facs pose for testing

[Thomas Larsson]

Should work now. Actually, there were three separate bugs.

1. The zero keys in the pose preset file were = 0, i.e. integers. In Blender, setting a property to an integer converts it to an integer property. Values are now converted to floats/integers depending on the existing property types.
2. Before loading the morphs, all properties were cleared, including integer properties like Eye Look Auto. Only floats should be cleared.
3. The pose preset contains a key for facs_ctrl_EyeBlink, but the property name is facs_ctrl_EyesBlink with an s. The association is made in an alias file, which the plugin ignores. The correct fix would be to load the alias files as well, but I am lazy and added a list of aliases to the plugin instead. If similar problems happen in the future, more aliases can be added to the list.

‌

[Alessandro Padovani]

Commit 59c302c seems to work great here. If Michele has nothing to add as for me we may mark as resolved.

[Thomas Larsson]

On second though, I didn’t like the idea to maintain an explicit list of aliases. It is error prone and will only work for standard morphs. A better solution is to load the aliasas at the same time that the morphs are loaded themselves. This is implemented in the last commit. Assuming that an alias file always has the same structure, this should solve the problem with aliases in a general way.

[Alessandro Padovani]

Commit 66b1ada seems fine here.