why is this 1

Author: CamDavidsonPilon

pioreactorui/api.py

@@ -1106,7 +1106,7 @@ def export_datasets() -> ResponseReturnValue:
         publish_to_error_log(msg, "export_datasets")
         return {"result": status, "filename": None, "msg": msg}, 500
 
-    if not status:
+    if status == b"false":
         publish_to_error_log(msg, "export_datasets")
         return {"result": status, "filename": None, "msg": msg}, 500
 
@@ -1740,15 +1740,15 @@ def setup_worker_pioreactor() -> ResponseReturnValue:
         return {"msg": str(e)}, 500
 
     try:
-        status, msg = result(blocking=True, timeout=250)
+        status = result(blocking=True, timeout=250)
     except HueyException:
-        status, msg = False, "Timed out, see logs."
+        status = False
     publish_to_log(status, "setup_worker_pioreactor")
+    publish_to_log(str(bool(status)), "setup_worker_pioreactor")
     if status:
         return {"msg": f"Worker {new_name} added successfully."}, 200
     else:
-        publish_to_error_log(msg, "setup_worker_pioreactor")
-        return {"msg": msg}, 500
+        return {"msg": f"Failed to add worker {new_name}"}, 500
 
 
 @api.route("/workers", methods=["PUT"])

pioreactorui/tasks.py

@@ -84,19 +84,14 @@ def pio_run(*args: str, env: dict[str, str] = {}) -> bool:
 
 
 @huey.task()
-def add_new_pioreactor(new_pioreactor_name: str, version: str, model: str) -> tuple[bool, str]:
+def add_new_pioreactor(new_pioreactor_name: str, version: str, model: str) -> bool:
     logger.info(f"Adding new pioreactor {new_pioreactor_name}, {model} {version}")
     result = run(
         [PIO_EXECUTABLE, "workers", "add", new_pioreactor_name, "-v", version, "-m", model],
-        capture_output=True,
-        text=True,
     )
     cache.evict("config")
     logger.info(result.returncode)
-    if result.returncode != 0:
-        return False, str(result.stderr.strip())
-    else:
-        return True, str(result.stderr.strip())
+    return result.returncode == 0
 
 
 @huey.task()