[Bf-committers] Revisions for the final 2.78: Rigify super_arm super_leg

[Sergey Sharybin]

Hi,

First two commtis are quite some code which does not address regression or
trivial fix. So it is rather really risky to accept it now.

The code itself is also weird:

- You can clearly simplify the layer mask creation, and don't have it
inlined as a list of False, False, False.... in multiple places
- Multiple subsequent try/catch could be replaced with a single check
whether `rigify_parameters` is in pbone. Using exceptions for that is
totally wrong!
- The goal is to have code pep8-compliant, and the new code is clearly not

Remaining two commits is clearly less code, but they are still suspicious.
Never heard of getting square of a transform matrix (mat*mat), and not sure
code was tested well enough after the change was done by all the users.

On Sat, Sep 24, 2016 at 3:16 PM, Lucio Rossi <lucio.rossi75 at gmail.com>
wrote:

> Hi,
> Adding to the last mail two commits that were missing...
>
> If it's possible include the following commits in the 2.78 release.
> They add a new super_arm and super_leg rigs with switchable child_of
> parenting:
>
> rBAff11927e8e07a46c8199d9681af639be52b6b1af
>
>
> rBAfecd5c9f566e0075e91d6d5830ccc0a622932736
>
> And as said in the previous mail:
>
> rBA816df6cfe1054acffad2ec23edd63ae504ab325f
>
> rBA4ed120fafb157aeeb9fbdb072a0e75375d658e24
>
> Lucio
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> https://lists.blender.org/mailman/listinfo/bf-committers
>



-- 
With best regards, Sergey Sharybin