Merge pull request #153 from BigRoy/master

Fix writeUVSet support for pointcache/alembic extraction and some better error/warning logging

colorbleed/maya/lib.py

@@ -76,6 +76,7 @@ _alembic_options = {
     "writeColorSets": bool,
     "writeFaceSets": bool,
     "writeCreases": bool,  # Maya 2015 Ext1+
+    "writeUVSets": bool,   # Maya 2017+
     "dataFormat": str,
     "root": (list, tuple),
     "attr": (list, tuple),
@@ -607,6 +608,8 @@ def extract_alembic(file,
 
         # Discard unknown options
         if key not in _alembic_options:
+            log.warning("extract_alembic() does not support option '%s'. "
+                        "Flag will be ignored..", key)
             options.pop(key)
             continue
 

colorbleed/plugins/maya/publish/extract_pointcache.py

@@ -35,11 +35,9 @@ class ExtractColorbleedAlembic(colorbleed.api.Extractor):
         # Get extra export arguments
         writeColorSets = instance.data.get("writeColorSets", False)
 
-        self.log.info("Extracting animation..")
+        self.log.info("Extracting pointcache..")
         dirname = self.staging_dir(instance)
 
-        self.log.info("nodes: %s" % str(nodes))
-
         parent_dir = self.staging_dir(instance)
         filename = "{name}.abc".format(**instance.data)
         path = os.path.join(parent_dir, filename)

colorbleed/plugins/maya/publish/validate_transfers.py

@@ -22,8 +22,6 @@ class ValidateTransfers(pyblish.api.InstancePlugin):
         if not transfers:
             return
 
-        verbose = instance.data.get('verbose', False)
-
         # Collect all destination with its sources
         collected = defaultdict(set)
         for source, destination in transfers:
@@ -39,16 +37,9 @@ class ValidateTransfers(pyblish.api.InstancePlugin):
             if len(sources) > 1:
                 invalid_destinations.append(destination)
 
-                if verbose:
-                    self.log.error("Non-unique file transfer for resources: "
-                                   "{0} (sources: {1})".format(destination,
-                                                               sources))
+                self.log.error("Non-unique file transfer for resources: "
+                               "{0} (sources: {1})".format(destination,
+                                                           list(sources)))
 
         if invalid_destinations:
-            if not verbose:
-                # If not verbose then still log the resource destination as
-                # opposed to every individual file transfer
-                self.log.error("Non-unique file transfers to destinations: "
-                               "%s" % "\n".join(invalid_destinations))
-
             raise RuntimeError("Invalid transfers in queue.")