Maya: abc options for Pointcache/Animation family - OP-5920

openpype/hosts/maya/plugins/publish/collect_animation.py

@@ -59,16 +59,28 @@ def process(self, instance):
             instance.data["families"].append("publish.farm")
 
         # Collect user defined attributes.
-        if not instance.data.get("includeUserDefinedAttributes", False):
-            return
-
-        user_defined_attributes = set()
-        for node in hierarchy:
-            attrs = cmds.listAttr(node, userDefined=True) or list()
-            shapes = cmds.listRelatives(node, shapes=True) or list()
-            for shape in shapes:
-                attrs.extend(cmds.listAttr(shape, userDefined=True) or list())
-
-            user_defined_attributes.update(attrs)
-
-        instance.data["userDefinedAttributes"] = list(user_defined_attributes)
+        if instance.data.get("includeUserDefinedAttributes", False):
+            user_defined_attributes = set()
+            for node in hierarchy:
+                attrs = cmds.listAttr(node, userDefined=True) or list()
+                shapes = cmds.listRelatives(node, shapes=True) or list()
+                for shape in shapes:
+                    attrs.extend(
+                        cmds.listAttr(shape, userDefined=True) or list()
+                    )
+
+                user_defined_attributes.update(attrs)
+
+            instance.data["userDefinedAttributes"] = list(
+                user_defined_attributes
+            )
+
+        # Backwards compatibility for attributes.
+        backwards_mapping = {
+            "write_color_sets": "writeColorSets",
+            "write_face_sets": "writeFaceSets",
+            "include_parent_hierarchy": "includeParentHierarchy",
+            "include_user_defined_attributes": "includeUserDefinedAttributes"
+        }
+        for key, value in backwards_mapping.items():
+            instance.data[value] = instance.data[key]

openpype/hosts/maya/plugins/publish/collect_pointcache.py

@@ -45,15 +45,26 @@ def process(self, instance):
 
         # Collect user defined attributes.
         if not instance.data.get("includeUserDefinedAttributes", False):
-            return
+            user_defined_attributes = set()
+            for node in instance:
+                attrs = cmds.listAttr(node, userDefined=True) or list()
+                shapes = cmds.listRelatives(node, shapes=True) or list()
+                for shape in shapes:
+                    attrs.extend(
+                        cmds.listAttr(shape, userDefined=True) or list()
+                    )
 
-        user_defined_attributes = set()
-        for node in instance:
-            attrs = cmds.listAttr(node, userDefined=True) or list()
-            shapes = cmds.listRelatives(node, shapes=True) or list()
-            for shape in shapes:
-                attrs.extend(cmds.listAttr(shape, userDefined=True) or list())
+                user_defined_attributes.update(attrs)
 
-            user_defined_attributes.update(attrs)
+            instance.data["userDefinedAttributes"] = list(
+                user_defined_attributes
+            )
 
-        instance.data["userDefinedAttributes"] = list(user_defined_attributes)
+        # Backwards compatibility for attributes.
+        backwards_mapping = {
+            "write_color_sets": "writeColorSets",
+            "write_face_sets": "writeFaceSets",
+            "include_user_defined_attributes": "includeUserDefinedAttributes"
+        }
+        for key, value in backwards_mapping.items():
+            instance.data[value] = instance.data[key]

openpype/hosts/maya/plugins/publish/extract_pointcache.py

@@ -81,10 +81,6 @@ def process(self, instance):
             # direct members of the set
             options["root"] = roots
 
-        if int(cmds.about(version=True)) >= 2017:
-            # Since Maya 2017 alembic supports multiple uv sets - write them.
-            options["writeUVSets"] = True
-
         if instance.data.get("visibleOnly", False):
             # If we only want to include nodes that are visible in the frame
             # range then we need to do our own check. Alembic's `visibleOnly`
@@ -96,8 +92,7 @@ def process(self, instance):
             )
 
         suspend = not instance.data.get("refresh", False)
-        self.log.info(nodes)
-        self.log.info(options)
+        self.log.debug(nodes)
         with suspended_refresh(suspend=suspend):
             with maintained_selection():
                 cmds.select(nodes, noExpand=True)