Welcome to mirror list, hosted at ThFree Co, Russian Federation.

git.blender.org/blender-addons.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexander Gavrilov <angavrilov@gmail.com>2019-10-15 15:33:10 +0300
committerAlexander Gavrilov <angavrilov@gmail.com>2019-10-15 15:36:32 +0300
commit0c859edf37b06de18bc0a6ac96c20ecbdc1b4052 (patch)
tree14a71fe0b95b6fe6ff522a32ee78b2db8fb95e02
parent0c21df4e49166d7aa0b7606db0f9e34ab93a7b68 (diff)
Rigify: fix incorrect layer assignment for tweak bones in the cat metarig.
The layer selection for the spine tweaks happens to be the default value, so simply copying the data does not overwrite non-default garbage values remaining in some of the bones. To fix it's necessary to clear params.
-rw-r--r--rigify/base_generate.py3
-rw-r--r--rigify/rigs/spines/super_spine.py2
-rw-r--r--rigify/utils/misc.py25
3 files changed, 27 insertions, 3 deletions
diff --git a/rigify/base_generate.py b/rigify/base_generate.py
index 8cf04ade..651e0612 100644
--- a/rigify/base_generate.py
+++ b/rigify/base_generate.py
@@ -27,7 +27,7 @@ from .utils.errors import MetarigError, RaiseErrorMixin
from .utils.naming import random_id
from .utils.metaclass import SingletonPluginMetaclass
from .utils.rig import list_bone_names_depth_first_sorted, get_rigify_type
-from .utils.misc import assign_parameters
+from .utils.misc import clone_parameters, assign_parameters
from . import base_rig
@@ -78,6 +78,7 @@ class SubstitutionRig(RaiseErrorMixin):
self.obj = generator.obj
self.base_bone = pose_bone.name
self.params = pose_bone.rigify_parameters
+ self.params_copy = clone_parameters(self.params)
def substitute(self):
# return [rig1, rig2...]
diff --git a/rigify/rigs/spines/super_spine.py b/rigify/rigs/spines/super_spine.py
index 3f7e41e1..5ed1588e 100644
--- a/rigify/rigs/spines/super_spine.py
+++ b/rigify/rigs/spines/super_spine.py
@@ -33,7 +33,7 @@ class Rig(SubstitutionRig, BoneUtilityMixin):
"""Compatibility proxy for the monolithic super_spine rig that splits it into parts."""
def substitute(self):
- params_copy = dict(self.params)
+ params_copy = self.params_copy
orgs = [self.base_bone] + connected_children_names(self.obj, self.base_bone)
# Split the bone list according to the settings
diff --git a/rigify/utils/misc.py b/rigify/utils/misc.py
index 4d0fbad3..64367bb7 100644
--- a/rigify/utils/misc.py
+++ b/rigify/utils/misc.py
@@ -24,6 +24,7 @@ import collections
from itertools import tee, chain, islice, repeat
from mathutils import Vector, Matrix, Color
+from rna_prop_ui import rna_idprop_value_to_python
#=============================================
@@ -170,8 +171,30 @@ def copy_attributes(a, b):
pass
+def property_to_python(value):
+ value = rna_idprop_value_to_python(value)
+
+ if isinstance(value, dict):
+ return { k: property_to_python(v) for k, v in value.items() }
+ elif isinstance(value, list):
+ return map_list(property_to_python, value)
+ else:
+ return value
+
+
+def clone_parameters(target):
+ return property_to_python(dict(target))
+
+
def assign_parameters(target, val_dict=None, **params):
- data = { **val_dict, **params } if val_dict else params
+ if val_dict is not None:
+ for key in list(target.keys()):
+ del target[key]
+
+ data = { **val_dict, **params }
+ else:
+ data = params
+
for key, value in data.items():
try:
target[key] = value