Option to install a particular version of jnml

OpenSourceBrain · Sep 1, 2023 · 81a4cec · 81a4cec
1 parent 4ff8207
commit 81a4cec
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -37,7 +37,7 @@ jobs:
           - PyLEMS
           - PyLEMS_NeuroML2
           - jLEMS
-          - jNeuroML
+          - "jNeuroML:v0.12.2"
           - jNeuroML_Brian2
           - jNeuroML_EDEN
           - "jNeuroML_NEURON:8.2.1"

diff --git a/omv/engines/getjnml.py b/omv/engines/getjnml.py
@@ -4,10 +4,13 @@
 from pathlib import Path
 
 from omv.engines.utils.wdir import working_dir
+from omv.common.inout import inform
 
 
-def install_jnml():
-    version = "v0.12.2"
+def install_jnml(version):
+
+    if not version:
+        version = "v0.12.2"
 
     try:
         jnmlhome = os.environ["JNML_HOME"]
@@ -41,3 +44,5 @@ def install_jnml():
             ]
         )
         check_output(["unzip", "jNeuroML.zip"])
+
+    inform("Successfully installed jNeuroML "+version, indent=1)
diff --git a/omv/omv_util.py b/omv/omv_util.py
@@ -226,14 +226,9 @@ def _install_engine(eng):
             if ee.is_installed():
                 already_installed = True
             else:
-                if engine_version is not None:
-                    raise Exception(
-                        "Currently, cannot install a specific version of engine %s"
-                        % eng
-                    )
                 from omv.engines.getjnml import install_jnml
 
-                install_jnml()
+                install_jnml(engine_version)
 
         elif eng.lower() == "neuroConstruct" or eng == "Py_neuroConstruct".lower():
             from omv.engines.pyneuroconstruct import PyneuroConstructEngine as ee