lamin_cli 1.5.2__tar.gz → 1.5.3__tar.gz

{lamin_cli-1.5.2 → lamin_cli-1.5.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: lamin_cli
-Version: 1.5.2
+Version: 1.5.3
 Summary: Lamin CLI.
 Author-email: Lamin Labs <open-source@lamin.ai>
 Description-Content-Type: text/markdown

lamin_cli-1.5.3/lamin_cli/__init__.py

@@ -0,0 +1,3 @@
+"""Lamin CLI."""
+
+__version__ = "1.5.3"

{lamin_cli-1.5.2 → lamin_cli-1.5.3}/lamin_cli/_save.py

@@ -6,6 +6,7 @@ from typing import TYPE_CHECKING
 
 import click
 from lamin_utils import logger
+from lamindb_setup.core.hashing import hash_file
 
 if TYPE_CHECKING:
     from pathlib import Path
@@ -155,6 +156,10 @@ def save_from_path_cli(
         return None
 
     if registry == "transform":
+        if key is not None:
+            logger.warning(
+                "key is ignored for transforms, the transform key is determined by the filename"
+            )
         if is_cloud_path:
             logger.error("Can not register a transform from a cloud path")
             return "transform-with-cloud-path"
@@ -201,8 +206,19 @@ def save_from_path_cli(
                 if transform is None:
                     uid = f"{stem_uid}0000"
         else:
-            # TODO: account for folders and hash equivalence as we do in ln.track()
+            # TODO: account for folders as we do in ln.track()
+            transform_hash, _ = hash_file(path)
             transform = ln.Transform.filter(key=path.name, is_latest=True).one_or_none()
+            if transform is not None and transform.hash is not None:
+                if transform.hash == transform_hash:
+                    logger.important(
+                        f"found existing Transform('{uid}') with matching hash"
+                    )
+                    return None
+                else:
+                    # we need to create a new version
+                    stem_uid = transform.uid[:12]
+                    transform = None
         revises = None
         if stem_uid is not None:
             revises = (

{lamin_cli-1.5.2 → lamin_cli-1.5.3}/tests/core/test_save_scripts.py

@@ -8,6 +8,27 @@ from lamindb_setup import settings
 scripts_dir = Path(__file__).parent.parent.resolve() / "scripts"
 
 
+def test_save_resave_script_no_uids():
+    filepath = scripts_dir / "testscript.py"
+    filepath.write_text("print('hello')")
+    result = subprocess.run(
+        f"lamin save {filepath}",
+        shell=True,
+        capture_output=True,
+    )
+    assert result.returncode == 0
+    assert "created Transform" in result.stdout.decode()
+    filepath.write_text("print('hello')\nprint('world')\n")
+    result = subprocess.run(
+        f"lamin save {filepath}",
+        shell=True,
+        capture_output=True,
+    )
+    assert result.returncode == 0
+    assert "created Transform" in result.stdout.decode()
+    assert ln.Transform.filter(key=filepath.name).count() == 2
+
+
 def test_save_without_uid():
     env = os.environ
     env["LAMIN_TESTING"] = "true"

lamin_cli-1.5.3/tests/scripts/testscript.py

@@ -0,0 +1,2 @@
+print("hello")
+print("world")

lamin_cli-1.5.2/lamin_cli/__init__.py

@@ -1,3 +0,0 @@
-"""Lamin CLI."""
-
-__version__ = "1.5.2"