Update get_input_var_names to return a list

R/.Rbuildignore

@@ -1,2 +1,4 @@
+^renv$
+^renv\.lock$
 ^LICENSE\.md$
 ^Dockerfile

python/pyproject.toml

@@ -37,7 +37,7 @@ dependencies = [
     # docker
     "docker",
 ]
-version = "0.1.0"
+version = "0.1.1"
 
 [project.optional-dependencies]
 dev = [
@@ -165,4 +165,4 @@ parallel = true
 omit = ["tests/*"]
 
 [tool.coverage.paths]
-source = ["src"]
+source = ["src"]

python/src/remotebmi/__init__.py

@@ -2,4 +2,4 @@
 from remotebmi.client.client import RemoteBmiClient
 from remotebmi.client.docker import BmiClientDocker
 
-__all__ = ["RemoteBmiClient", "BmiClientApptainer", "BmiClientDocker"]
+__all__ = ["BmiClientApptainer", "BmiClientDocker", "RemoteBmiClient"]

python/src/remotebmi/client/apptainer.py

@@ -9,6 +9,8 @@
 from remotebmi.client.client import RemoteBmiClient
 from remotebmi.client.utils import DeadContainerError, get_unique_port
 
+logger = logging.getLogger(__name__)
+
 
 class BmiClientApptainer(RemoteBmiClient):
     def __init__(
@@ -44,7 +46,7 @@ def __init__(
         # Change into working directory
         args += ["--pwd", self.work_dir]
         args.append(image)
-        logging.info(f"Running {image} apptainer container on port {port}")
+        logger.info(f"Running {image} apptainer container on port {port}")
         if capture_logs:
             self.logfile = SpooledTemporaryFile(  # noqa: SIM115 - file is closed in __del__
                 max_size=2**16,  # keep until 65Kb in memory if bigger write to disk

python/src/remotebmi/client/docker.py

@@ -4,7 +4,7 @@
 from posixpath import abspath
 
 import docker
-from docker.models.containers import Container  # noqa: TCH002
+from docker.models.containers import Container  # noqa: TC002
 
 from remotebmi.client.client import RemoteBmiClient
 from remotebmi.client.utils import DeadContainerError, get_unique_port, getuser

python/src/remotebmi/server/__init__.py

@@ -47,7 +47,7 @@ async def lifespan_handler(app: ConnexionMiddleware) -> AsyncIterator:  # noqa:
 def main(**kwargs):  # type: ignore[no-untyped-def]
     model = from_env()
     app = make_app(model)
-    port = int(environ.get("BMI_PORT", 50051))
+    port = int(environ.get("BMI_PORT", "50051"))
     uvicorn.run(app, port=port, **kwargs)
 
 

python/src/remotebmi/server/api.py

@@ -41,8 +41,9 @@ def get_component_name() -> dict[str, str]:
     return {"name": model().get_component_name()}
 
 
-def get_input_var_names() -> tuple[str, ...]:
-    return model().get_input_var_names()
+def get_input_var_names() -> list[str]:
+    n = model().get_input_var_names()
+    return list(n)
 
 
 def get_output_var_names() -> list[str]: