calkit-python 0.26.3__tar.gz → 0.26.5__tar.gz

{calkit_python-0.26.3 → calkit_python-0.26.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: calkit-python
-Version: 0.26.3
+Version: 0.26.5
 Summary: Reproducibility simplified.
 Project-URL: Homepage, https://calkit.org
 Project-URL: Issues, https://github.com/calkit/calkit/issues

{calkit_python-0.26.3 → calkit_python-0.26.5}/calkit/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "0.26.3"
+__version__ = "0.26.5"
 
 from .core import *  # noqa: F403, I001
 from . import git  # noqa: F401

{calkit_python-0.26.3 → calkit_python-0.26.5}/calkit/cli/check.py

@@ -26,7 +26,7 @@ import calkit
 import calkit.matlab
 import calkit.pipeline
 from calkit.check import check_reproducibility
-from calkit.cli import raise_error
+from calkit.cli import raise_error, warn
 
 check_app = typer.Typer(no_args_is_help=True)
 
@@ -466,9 +466,13 @@ def check_venv(
     lock_dir = os.path.dirname(lock_fpath)
     if lock_dir:
         os.makedirs(lock_dir, exist_ok=True)
+    # If the lock file exists, try to install with that
+    dep_file_txt = f"-r {path}"
+    if os.path.isfile(lock_fpath):
+        dep_file_txt += f" -r {lock_fpath}"
     check_cmd = (
         f"{activate_cmd} "
-        f"&& {pip_cmd} install {pip_install_args} -r {path} "
+        f"&& {pip_cmd} install {pip_install_args} {dep_file_txt} "
         f"&& {pip_freeze_cmd} > {lock_fpath} "
         "&& deactivate"
     )
@@ -482,7 +486,18 @@ def check_venv(
             stderr=subprocess.STDOUT if not verbose else None,
         )
     except subprocess.CalledProcessError:
-        raise_error(f"Failed to check {kind}")
+        warn("Failed to create environment from lock file; attempting rebuild")
+        check_cmd = (
+            f"{activate_cmd} "
+            f"&& {pip_cmd} install {pip_install_args} -r {path} "
+            f"&& {pip_freeze_cmd} > {lock_fpath} "
+            "&& deactivate"
+        )
+        # Try again and don't capture output
+        try:
+            subprocess.run(check_cmd, shell=True, cwd=wdir, check=True)
+        except subprocess.CalledProcessError:
+            raise_error(f"Failed to check {kind}")
 
 
 @check_app.command(name="matlab-env")

{calkit_python-0.26.3 → calkit_python-0.26.5}/calkit/cli/main.py

@@ -874,7 +874,7 @@ def run(
     ] = False,
     no_log: Annotated[
         bool,
-        typer.Option("--no-log", help="Do not log the run."),
+        typer.Option("--no-log", "-l", help="Do not log the run."),
     ] = False,
     save_after_run: Annotated[
         bool,
@@ -893,16 +893,17 @@ def run(
     if not quiet:
         typer.echo("Getting system information")
     system_info = calkit.get_system_info()
-    # Save the system to .calkit/systems
-    if verbose:
-        typer.echo("Saving system information:")
-        typer.echo(system_info)
-    sysinfo_fpath = os.path.join(
-        ".calkit", "systems", system_info["id"] + ".json"
-    )
-    os.makedirs(os.path.dirname(sysinfo_fpath), exist_ok=True)
-    with open(sysinfo_fpath, "w") as f:
-        json.dump(system_info, f, indent=2)
+    if not no_log:
+        # Save the system to .calkit/systems
+        if verbose:
+            typer.echo("Saving system information:")
+            typer.echo(system_info)
+        sysinfo_fpath = os.path.join(
+            ".calkit", "systems", system_info["id"] + ".json"
+        )
+        os.makedirs(os.path.dirname(sysinfo_fpath), exist_ok=True)
+        with open(sysinfo_fpath, "w") as f:
+            json.dump(system_info, f, indent=2)
     # First check any system-level dependencies exist
     if not quiet:
         typer.echo("Checking system-level dependencies")
@@ -1189,6 +1190,12 @@ def run_in_env(
                 ),
                 platform=env.get("platform"),
                 deps=env.get("deps", []),
+                env_vars=env.get("env_vars", []),
+                ports=env.get("ports", []),
+                gpus=env.get("gpus"),
+                user=env.get("user"),
+                wdir=env.get("wdir"),
+                args=env.get("args", []),
                 quiet=not verbose,
             )
         shell_cmd = _to_shell_cmd(cmd)

{calkit_python-0.26.3 → calkit_python-0.26.5}/calkit/cli/new.py

@@ -133,6 +133,11 @@ def new_project(
             raise_error(
                 f"Could not detect Git repo URL from existing repo: {e}"
             )
+        # We don't have a project name but do have a Git repo URL, the
+        # project name will be the Git repo name lowercased, since that's
+        # how the Calkit Cloud will create the project name by default
+        if name is None and git_repo_url is not None:
+            name = git_repo_url.split("/")[-1].lower()
         # If this isn't a DVC repo, run `dvc init`
         if not os.path.isfile(os.path.join(abs_path, ".dvc", "config")):
             typer.echo("Initializing DVC repository")

{calkit_python-0.26.3 → calkit_python-0.26.5}/calkit/conda.py

@@ -17,9 +17,21 @@ from calkit import ryaml
 
 
 def _check_single(req: str, actual: str, conda: bool = False) -> bool:
-    """Helper function for checking actual versions against requirements."""
-    req_name = re.split("[=<>]", req)[0]
-    req_spec = req.removeprefix(req_name)
+    """Helper function for checking actual versions against requirements.
+
+    Note that this also doesn't check optional dependencies.
+    """
+    # If this is a Git version, we can't check it
+    # TODO: Clone Git repos to check?
+    if "@git" in req:
+        warnings.warn(f"Cannot check Git version for {req}")
+        req = req.split("@git")[0]
+    req_name = re.split("[=<>]", req)[0].strip()
+    req_spec = req.removeprefix(req_name).strip().replace(" ", "")
+    if "[" in req_name:
+        warnings.warn(f"Cannot check optional dependencies for {req_name}")
+        # Remove optional dependencies
+        req_name = req_name.split("[")[0].strip()
     if conda and req_spec.startswith("="):
         req_spec = "=" + req_spec
         if not req_spec.endswith(".*"):

calkit_python-0.26.5/calkit/tests/cli/test_check.py

@@ -0,0 +1,45 @@
+"""Tests for ``calkit.cli.check``."""
+
+import shutil
+import subprocess
+
+
+def test_check_venv(tmp_dir):
+    with open("reqs.txt", "w") as f:
+        f.write("requests")
+    subprocess.check_call(
+        ["calkit", "check", "venv", "reqs.txt", "-o", "lock.txt"]
+    )
+    # Now check that we can install from the lock file
+    shutil.rmtree(".venv")
+    subprocess.check_call(
+        ["calkit", "check", "venv", "reqs.txt", "-o", "lock.txt"]
+    )
+    with open("lock.txt") as f:
+        lock_txt = f.read()
+    # Now check that if we add a requirement, the env is rebuilt
+    assert "polars" not in lock_txt
+    with open("reqs.txt", "w") as f:
+        f.write("requests\npolars")
+    subprocess.check_call(
+        ["calkit", "check", "venv", "reqs.txt", "-o", "lock.txt"]
+    )
+    with open("lock.txt") as f:
+        lock_txt = f.read()
+    assert "polars" in lock_txt
+    # Now confirm that if we check the env again, nothing happens
+    subprocess.check_call(
+        ["calkit", "check", "venv", "reqs.txt", "-o", "lock.txt"]
+    )
+    with open("lock.txt") as f:
+        lock_txt_2 = f.read()
+    assert lock_txt == lock_txt_2
+    # Now check that if we pin a version in reqs.txt, we rebuild
+    with open("reqs.txt", "w") as f:
+        f.write("requests\npolars==1.0.0")
+    subprocess.check_call(
+        ["calkit", "check", "venv", "reqs.txt", "-o", "lock.txt"]
+    )
+    with open("lock.txt") as f:
+        lock_txt_3 = f.read()
+    assert "polars==1.0.0" in lock_txt_3