planetarypy 0.53.2__tar.gz → 0.53.4__tar.gz

{planetarypy-0.53.2 → planetarypy-0.53.4}/CHANGELOG.md

@@ -5,6 +5,19 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.53.4] - 2026-04-24
+
+### Fixed
+- **`url_retrieve` is now concurrency-safe.** Previously two processes (e.g. parallel pytest-xdist workers hitting `load_generic_kernels()` via `Spicer("MARS")` / `Spicer("MOON")`) could clobber each other's `.part` scratch file and race on the final `rename()`, producing `FileNotFoundError`. The scratch file now includes the writer's PID (`{name}.{pid}.part`), and when a concurrent winner has already moved the final file into place the loser silently drops its scratch copy instead of raising. This was the root cause of intermittent CI failures in `test_spicer`.
+
+### Changed
+- CI workflow `test.yaml` now prefetches the SPICE generic kernels in a single-writer step before invoking `pytest`, so parallel test workers see cached files and never trigger the download path simultaneously. Complements the `url_retrieve` fix; either alone would green CI, both together harden the library for any concurrent caller.
+
+## [0.53.3] - 2026-04-24
+
+### Fixed
+- **`plp fetch` and `plp hibrowse` now emit only the resolved file path on stdout.** Diagnostic lines ("Resolving…", "URL:…", "Fetching…") and the "Browse:" prefix previously mixed with the final path on stdout, which made shell command substitution clumsy (e.g. `qgis (plp fetch mro.ctx.edr <pid>)` captured all of it as arguments). Diagnostics now go to stderr; only the payload path hits stdout.
+
 ## [0.53.2] - 2026-04-24
 
 ### Fixed

{planetarypy-0.53.2 → planetarypy-0.53.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: planetarypy
-Version: 0.53.2
+Version: 0.53.4
 Summary: Core package for planetary data tools. Includes PDS index utilities, SPICE integrations, and more.
 Project-URL: Homepage, https://github.com/planetarypy/planetarypy
 Project-URL: Repository, https://github.com/planetarypy/planetarypy

{planetarypy-0.53.2 → planetarypy-0.53.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "planetarypy"
-version = "0.53.2"
+version = "0.53.4"
 description = "Core package for planetary data tools. Includes PDS index utilities, SPICE integrations, and more."
 readme = "README.md"
 requires-python = ">= 3.11, <4"
@@ -127,6 +127,6 @@ max-line-length = 88
 extend-ignore = ["E203", "E701"]
 
 [tool.bumpversion]
-current_version = "0.53.2"
+current_version = "0.53.4"
 commit = true
 tag = true

{planetarypy-0.53.2 → planetarypy-0.53.4}/src/planetarypy/cli.py

@@ -64,11 +64,11 @@ def fetch(
 
     mission, instrument, product_key = key.split(".")
 
-    typer.echo(f"Resolving {key} / {product_id}...")
+    typer.echo(f"Resolving {key} / {product_id}...", err=True)
     try:
         resolved = resolve_product(mission, instrument, product_key, product_id)
         for f in resolved.files:
-            typer.echo(f"URL: {resolved.url_stem}/{f}")
+            typer.echo(f"URL: {resolved.url_stem}/{f}", err=True)
         if here:
             local_dir = Path.cwd()
         else:
@@ -76,6 +76,9 @@ def fetch(
                 mission, instrument, product_key, resolved.product_id,
             )
         download_product(resolved, local_dir, label_only=label_only, force=force)
+        # Final resolved paths go to stdout (and only stdout) so that
+        # shell command substitution — e.g. `qgis (plp fetch …)` —
+        # captures just the file paths, not the diagnostic chatter.
         for f in resolved.files:
             typer.echo(local_dir / f)
     except Exception as e:
@@ -127,12 +130,17 @@ def hibrowse(
     obs_id = f"{parts[0]}_{parts[1]}_{parts[2]}"
     orbit = int(parts[1])
     suffix = "abrowse.jpg" if annotated else "browse.jpg"
-    typer.echo(f"Fetching {HIRISE_BASE}/EXTRAS/{data_level}/{parts[0]}/{_orbit_range(orbit)}/{obs_id}/{pid}.{suffix}")
+    typer.echo(
+        f"Fetching {HIRISE_BASE}/EXTRAS/{data_level}/{parts[0]}/{_orbit_range(orbit)}/{obs_id}/{pid}.{suffix}",
+        err=True,
+    )
 
     try:
         dest = Path.cwd() if here else None
         outpath = get_browse(product_id, annotated=annotated, dest=dest, force=force)
-        typer.echo(f"Browse: {outpath}")
+        # Raw path on stdout so `qgis (plp hibrowse …)` and similar
+        # shell substitutions capture just the path.
+        typer.echo(outpath)
     except Exception as e:
         typer.echo(f"Error: {e}", err=True)
         raise typer.Exit(1)

{planetarypy-0.53.2 → planetarypy-0.53.4}/src/planetarypy/utils.py

@@ -2,6 +2,7 @@
 import datetime as dt
 import email.utils as eut
 import http.client as httplib
+import os
 from pathlib import Path
 from typing import Any
 from urllib.request import Request, urlopen
@@ -276,7 +277,10 @@ def url_retrieve(
     """
     url = str(url)
     outfile = Path(outfile)
-    part_file = outfile.with_suffix(outfile.suffix + ".part")
+    # Per-PID scratch file so concurrent callers (e.g. parallel pytest
+    # workers, multiprocessing) don't clobber each other's partial
+    # writes and don't race on the final rename.
+    part_file = outfile.with_suffix(f"{outfile.suffix}.{os.getpid()}.part")
 
     if user:
         auth = HTTPBasicAuth(user, passwd)
@@ -301,7 +305,19 @@ def url_retrieve(
     ) as fd:
         for chunk in R.iter_content(chunk_size=chunk_size):
             fd.write(chunk)
-    part_file.rename(outfile)
+    # If another concurrent writer already finished first, drop our
+    # scratch file rather than overwriting the winner.
+    if outfile.exists():
+        part_file.unlink(missing_ok=True)
+    else:
+        try:
+            part_file.replace(outfile)
+        except FileNotFoundError:
+            # Lost a very tight race: another writer moved our part
+            # file out from under us (unlikely with per-PID suffix).
+            # If outfile now exists we're still fine.
+            if not outfile.exists():
+                raise
 
 
 def have_internet() -> bool: