Allow to process legacy conversation data without row_data

Author: Val Brodsky

libs/labelbox/src/labelbox/schema/internal/data_row_uploader.py

@@ -20,7 +20,6 @@ def upload_in_chunks(client, specs: List[DataRowItemBase],
                      file_upload_thread_count: int,
                      max_chunk_size_bytes: int) -> UploadManifest:
     empty_specs = list(filter(lambda spec: spec.is_empty(), specs))
-
     if empty_specs:
         ids = list(map(lambda spec: spec.id.get("value"), empty_specs))
         ids = list(filter(lambda x: x is not None and len(x) > 0, ids))

libs/labelbox/src/labelbox/schema/internal/data_row_upsert_item.py

@@ -81,9 +81,15 @@ def is_empty(self) -> bool:
         """
         row_data = self.payload.get("row_data", None) or self.payload.get(
             DataRow.row_data, None)
-        return (not self.payload or len(self.payload.keys()) == 1 and
-                "dataset_id" in self.payload or row_data is None or
-                len(row_data) == 0)
+
+        return (not self._is_legacy_conversational_data() and
+                (not self.payload or len(self.payload.keys()) == 1 and
+                 "dataset_id" in self.payload or row_data is None or
+                 len(row_data) == 0))
+
+    def _is_legacy_conversational_data(self) -> bool:
+        return "conversationalData" in self.payload.keys(
+        ) or "conversational_data" in self.payload.keys()
 
     @classmethod
     def build(

libs/labelbox/tests/unit/test_data_row_upsert_data.py

@@ -125,6 +125,25 @@ def test_create_is_empty():
         payload={DataRow.row_data: "http://my_site.com/photos/img_01.jpg"})
     assert not item.is_empty()
 
+    legacy_converstational_data_payload = {
+        "externalId":
+            "Convo-123",
+        "type":
+            "application/vnd.labelbox.conversational",
+        "conversationalData": [{
+            "messageId":
+                "message-0",
+            "content":
+                "I love iphone! i just bought new iphone! :smiling_face_with_3_hearts: :calling:",
+            "user": {
+                "userId": "Bot 002",
+                "name": "Bot"
+            },
+        }]
+    }
+    item = DataRowCreateItem(id={}, payload=legacy_converstational_data_payload)
+    assert not item.is_empty()
+
 
 def test_create_row_data_none():
     items = [