fix: assign columns instead of modifying input slide manifest

msk-mind · Nov 2, 2023 · af76b7a · af76b7a
1 parent ba01836
commit af76b7a
Show file tree

Hide file tree

Showing 8 changed files with 212 additions and 168 deletions.
diff --git a/src/luna/pathology/cli/dsa_upload.py b/src/luna/pathology/cli/dsa_upload.py
@@ -136,6 +136,7 @@ def upload_annotation_to_dsa(
     Returns:
         DataFrame[SlideSchema]: slide manifest
     """
+    uuids = []
     for slide in slide_manifest.itertuples(name="Slide"):
         uuids = _upload_annotation_to_dsa(
             dsa_endpoint_url,
@@ -148,10 +149,8 @@ def upload_annotation_to_dsa(
             insecure,
             storage_options,
         )
-        slide_manifest.at[
-            slide.Index, annotation_column.replace("url", "uuid")
-        ] = uuids[0]
-    return slide_manifest
+        uuids.append(uuids[0])
+    return slide_manifest.assign(**{annotation_column: uuids})
 
 
 def _upload_annotation_to_dsa(