remove print statement and add comments

mesoscope · Nov 4, 2024 · e82ca6f · e82ca6f
1 parent 19d556c
commit e82ca6f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/cellpack/autopack/upy/simularium/simularium_helper.py b/cellpack/autopack/upy/simularium/simularium_helper.py
@@ -1399,8 +1399,6 @@ def post_and_open_file(self, file_name, open_results_in_browser=True):
         simularium_file = Path(f"{file_name}.simularium")
         url = None
         job_id = os.environ.get("AWS_BATCH_JOB_ID", None)
-        if job_id:
-            print(f"Batch Job ID: {job_id}")
         file_name, url = simulariumHelper.store_result_file(
             simularium_file, storage="aws", batch_job_id=job_id
         )
@@ -1415,7 +1413,9 @@ def post_and_open_file(self, file_name, open_results_in_browser=True):
     def store_result_file(file_path, storage=None, batch_job_id=None):
         if storage == "aws":
             handler = DATABASE_IDS.handlers().get(storage)
-            if batch_job_id:  # save results to batch job bucket
+            # if batch_job_id is not None, then we are in a batch job and should use the temp bucket
+            # TODO: use cellpack-results bucket for batch jobs once we have the correct permissions
+            if batch_job_id:
                 initialized_handler = handler(
                     bucket_name="cellpack-demo",
                     sub_folder_name="simularium",