simulo 0.7.1__tar.gz → 0.8.0__tar.gz

{simulo-0.7.1/src/simulo.egg-info → simulo-0.8.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: simulo
-Version: 0.7.1
+Version: 0.8.0
 Summary: Simulo SDK and CLI — define robotics simulation and training apps in Python and run them on the Simulo cloud.
 Author-email: Simulo Team <team@simulo.ai>
 License: BSD-3-Clause
@@ -16,6 +16,7 @@ Classifier: Typing :: Typed
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 Requires-Dist: simulo-interfaces<0.2,>=0.1
+Requires-Dist: mcap<2,>=1.3
 Provides-Extra: dev
 Requires-Dist: black>=24.0; extra == "dev"
 Requires-Dist: isort>=5.13; extra == "dev"

{simulo-0.7.1 → simulo-0.8.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "simulo"
-version = "0.7.1"
+version = "0.8.0"
 description = "Simulo SDK and CLI — define robotics simulation and training apps in Python and run them on the Simulo cloud."
 readme = "PYPI.md"
 requires-python = ">=3.11"
@@ -20,10 +20,12 @@ classifiers = [
     "Programming Language :: Python :: 3.12",
     "Typing :: Typed",
 ]
-# Torch-free by construction: depends only on the contract layer. Heavy
-# imports (e.g. torch) belong inside the user's `runtime.imports()` guard,
-# resolved only at job-execution time — never in this distribution.
-dependencies = ["simulo-interfaces>=0.1,<0.2"]
+# Torch-free by construction: depends only on the contract layer and mcap.
+# Heavy imports (e.g. torch) belong inside the user's `runtime.imports()`
+# guard, resolved only at job-execution time — never in this distribution.
+# mcap: used by `simulo recordings` to read back a downloaded recording and
+# verify it (message count) — MIT-licensed, deps only lz4+zstandard.
+dependencies = ["simulo-interfaces>=0.1,<0.2", "mcap>=1.3,<2"]
 
 [project.optional-dependencies]
 dev = ["black>=24.0", "isort>=5.13", "ruff>=0.5", "mypy>=1.10", "pytest>=8.2", "build>=1.2", "twine>=5.1"]

{simulo-0.7.1 → simulo-0.8.0}/src/simulo/_client/cli.py

@@ -450,8 +450,36 @@ def _recording_name(record: dict[str, Any]) -> str:
     return str(name) if name else f"{record.get('recording_id', 'recording')}.mcap"
 
 
+def _verify_recording(dest: Path) -> None:
+    """Read *dest* back with the ``mcap`` reader and print a one-line summary.
+
+    This is the genuine use of the bundled ``mcap`` dependency (pyproject
+    dependencies): it replaces the manual ``python -c "from mcap.reader
+    import ..."`` check a user previously had to run by hand after `simulo
+    recordings` finished. The import is LAZY (inside this function, not at
+    module scope) so a plain `import simulo` — and every other CLI command
+    that never touches recordings — stays fast/torch-free; `mcap` is light,
+    but lazy is still the right default for a CLI-only code path.
+
+    Non-fatal by design: a corrupt or truncated download must not turn an
+    otherwise-successful fetch (the bytes are already on disk) into a crash.
+    Any failure — a genuinely corrupt file, or an mcap-library edge case —
+    prints a WARNING to stderr and returns; the file is kept and the caller's
+    exit code is unaffected.
+    """
+    try:
+        from mcap.reader import make_reader
+
+        with open(dest, "rb") as f:
+            reader = make_reader(f)
+            message_count = sum(1 for _ in reader.iter_messages())
+        print(f"verified {dest.name} — {message_count} messages", file=sys.stderr)
+    except Exception as exc:  # noqa: BLE001 — best-effort verification only; must never crash a successful download
+        print(f"WARNING: could not verify {dest.name} as a valid MCAP file: {exc}", file=sys.stderr)
+
+
 def _download_and_report(client: SubmitApiClient, job_id: str, record: dict[str, Any], dest: Path) -> bool:
-    """Download one recording to *dest*; print the Lichtblick hint on success.
+    """Download one recording to *dest*; verify + print the Lichtblick hint on success.
 
     ``dest`` must already be a caller-vetted destination — either the
     user's own ``-o PATH`` or ``out_dir / _safe_download_filename(...)``
@@ -467,12 +495,17 @@ def _download_and_report(client: SubmitApiClient, job_id: str, record: dict[str,
     — raises bare ``ValueError`` from the filesystem call, and it deserves
     the same clean one-liner (server-supplied names can no longer get here;
     ``_safe_download_filename`` rejects NULs outright).
+
+    A successful download is read back and verified (``_verify_recording``,
+    non-fatal) before the Lichtblick hint prints — a corrupt download is
+    still reported as a WARNING, never a crash.
     """
     try:
         resolved = client.download_recording(job_id, str(record["recording_id"]), dest)
     except (OSError, ValueError) as exc:
         print(f"Could not write recording to {dest}: {exc}", file=sys.stderr)
         return False
+    _verify_recording(resolved)
     print(f"Open in Lichtblick: {resolved.resolve()}")
     return True
 

{simulo-0.7.1 → simulo-0.8.0/src/simulo.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: simulo
-Version: 0.7.1
+Version: 0.8.0
 Summary: Simulo SDK and CLI — define robotics simulation and training apps in Python and run them on the Simulo cloud.
 Author-email: Simulo Team <team@simulo.ai>
 License: BSD-3-Clause
@@ -16,6 +16,7 @@ Classifier: Typing :: Typed
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 Requires-Dist: simulo-interfaces<0.2,>=0.1
+Requires-Dist: mcap<2,>=1.3
 Provides-Extra: dev
 Requires-Dist: black>=24.0; extra == "dev"
 Requires-Dist: isort>=5.13; extra == "dev"

{simulo-0.7.1 → simulo-0.8.0}/src/simulo.egg-info/requires.txt

@@ -1,4 +1,5 @@
 simulo-interfaces<0.2,>=0.1
+mcap<2,>=1.3
 
 [dev]
 black>=24.0