chore: reduce logging level of some errors

Author: sanjayankur31

pyneuroml/runners.py

@@ -922,7 +922,7 @@ def execute_command_in_dir_with_realtime_output(
         raise e
 
     if not p.returncode == 0:
-        logger.critical(
+        logger.error(
             "*** Problem running command (return code: %s): [%s]"
             % (p.returncode, command)
         )
@@ -993,14 +993,14 @@ def execute_command_in_dir(
         return return_string.decode("utf-8")
 
     except subprocess.CalledProcessError as e:
-        logger.critical("*** Problem running last command: %s" % e)
+        logger.error("*** Problem running last command: %s" % e)
 
         print("####################################################################")
         print("%s%s" % (prefix, e.output.decode().replace("\n", "\n" + prefix)))
         print("####################################################################")
         return (e.returncode, e.output.decode())
     except Exception as e:
-        logger.critical("*** Unknown problem running command: %s" % e)
+        logger.error("*** Unknown problem running command: %s" % e)
         return (-1, str(e))
 
 

pyneuroml/utils/jnmlwrapper.py

@@ -36,10 +36,13 @@ def __jnmlwrapper():
         report_jnml_output=False,
         output_prefix="",
         return_string=True,
-        exit_on_fail=True,
+        exit_on_fail=False,
     )
 
     # if command ran successfully, print the output
     # if it didn't, `run_jneuroml` will throw a critical error
     if retstat is True:
         print(output)
+        sys.exit(0)
+    else:
+        sys.exit(1)