TVPaint: Publish workfile.

openpype/hosts/tvpaint/plugins/publish/collect_instances.py

@@ -1,3 +1,4 @@
+import os
 import json
 import copy
 import pyblish.api
@@ -17,6 +18,8 @@ def process(self, context):
             json.dumps(workfile_instances, indent=4)
         ))
 
+        self.add_workfile_instance(context)
+
         # Backwards compatibility for workfiles that already have review
         #   instance in metadata.
         review_instance_exist = False
@@ -109,7 +112,7 @@ def _create_review_instance_data(self, context):
 
         return {
             "family": "review",
-            "asset": context.data["workfile_context"]["asset"],
+            "asset": context.data["asset"],
             # Dummy subset name
             "subset": "reviewMain"
         }
@@ -230,3 +233,36 @@ def create_render_pass_instance(self, context, instance_data):
 
         instance_data["layers"] = render_pass_layers
         return context.create_instance(**instance_data)
+
+    def add_workfile_instance(self, context):
+        current_file = context.data["currentFile"]
+
+        self.log.info(
+            "Workfile path used for workfile family: {}".format(current_file)
+        )
+
+        dirpath, filename = os.path.split(current_file)
+        basename, ext = os.path.splitext(filename)
+        instance = context.create_instance(name=basename)
+
+        task_name = io.Session["AVALON_TASK"]
+        subset_name = "workfile" + task_name.capitalize()
+
+        # Create Workfile instance
+        instance.data.update({
+            "subset": subset_name,
+            "asset": context.data["asset"],
+            "label": subset_name,
+            "publish": True,
+            "family": "workfile",
+            "families": ["workfile"],
+            "representations": [{
+                "name": ext.lstrip("."),
+                "ext": ext.lstrip("."),
+                "files": filename,
+                "stagingDir": dirpath
+            }]
+        })
+        self.log.info("Collected workfile instance: {}".format(
+            json.dumps(instance.data, indent=4)
+        ))

openpype/hosts/tvpaint/plugins/publish/validate_workfile_metadata.py

@@ -0,0 +1,49 @@
+import pyblish.api
+from avalon.tvpaint import save_file
+
+
+class ValidateWorkfileMetadataRepair(pyblish.api.Action):
+    """Store current context into workfile metadata."""
+
+    label = "Use current context"
+    icon = "wrench"
+    on = "failed"
+
+    def process(self, context, _plugin):
+        """Save current workfile which should trigger storing of metadata."""
+        current_file = context.data["currentFile"]
+        # Save file should trigger
+        save_file(current_file)
+
+
+class ValidateWorkfileMetadata(pyblish.api.ContextPlugin):
+    """Validate if wokrfile contain required metadata for publising."""
+
+    label = "Validate Workfile Metadata"
+    order = pyblish.api.ValidatorOrder
+
+    families = ["workfile"]
+
+    actions = [ValidateWorkfileMetadataRepair]
+
+    required_keys = {"project", "asset", "task"}
+
+    def process(self, context):
+        workfile_context = context.data["workfile_context"]
+        if not workfile_context:
+            raise AssertionError(
+                "Current workfile is missing whole metadata about context."
+            )
+
+        missing_keys = []
+        for key in self.required_keys:
+            value = workfile_context.get(key)
+            if not value:
+                missing_keys.append(key)
+
+        if missing_keys:
+            raise AssertionError(
+                "Current workfile is missing metadata about {}.".format(
+                    ", ".join(missing_keys)
+                )
+            )

openpype/plugins/publish/extract_review.py

@@ -1686,7 +1686,7 @@ def video_filters(self):
     def _convert_string_to_values(self, orig_string_value):
         string_value = orig_string_value.strip().lower()
         if not string_value:
-            return
+            return [PixValueRelative(0), PixValueRelative(0)]
 
         # Replace "px" (and spaces before) with single space
         string_value = re.sub(r"([ ]+)?px", " ", string_value)