Merge pull request #64 from scaleapi/da/validation_serialization

Minor bugfixes

Author: ardila

nucleus/dataset.py

@@ -241,9 +241,9 @@ def create_slice(
 
         :return: new Slice object
         """
-        if dataset_item_ids and reference_ids:
+        if bool(dataset_item_ids) == bool(reference_ids):
             raise Exception(
-                "You cannot specify both dataset_item_ids and reference_ids"
+                "You must specify exactly one of dataset_item_ids or reference_ids."
             )
         payload: Dict[str, Any] = {NAME_KEY: name}
         if dataset_item_ids:

nucleus/utils.py

@@ -77,7 +77,7 @@ def serialize_and_write(
 ):
     for unit in upload_unit:
         try:
-            file_pointer.write(unit.to_json())
+            file_pointer.write(unit.to_json() + "\n")
         except TypeError as e:
             type_name = type(unit).__name__
             message = (

pyproject.toml

@@ -21,7 +21,7 @@ exclude = '''
 
 [tool.poetry]
 name = "scale-nucleus"
-version = "0.1.2"
+version = "0.1.3"
 description = "The official Python client library for Nucleus, the Data Platform for AI"
 license =  "MIT"
 authors = ["Scale AI Nucleus Team <nucleusapi@scaleapi.com>"]
@@ -32,7 +32,7 @@ documentation = "https://dashboard.scale.com/nucleus/docs/api"
 packages = [{include="nucleus"}]
 
 [tool.poetry.dependencies]
-python = "^3.6.1"
+python = "^3.6.2"
 grequests = "^0.6.0"
 requests = "^2.25.1"
 tqdm = "^4.60.0"