paces 0.0.20__tar.gz → 0.0.21__tar.gz

{paces-0.0.20 → paces-0.0.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: paces
-Version: 0.0.20
+Version: 0.0.21
 Summary: Turn instructional media into structured, interactive learning material
 Project-URL: Homepage, https://github.com/thorwhalen/paces
 Project-URL: Repository, https://github.com/thorwhalen/paces

{paces-0.0.20 → paces-0.0.21}/paces/measure.py

@@ -231,7 +231,7 @@ def measure_grid(
         flags += (f"tempo-unmeasured: {tempo_unmeasured_note}{suffix}",)
 
     # Explicit beats inferred — but a disagreement is a finding, not a secret.
-    final_tempo = known_tempo or measured_tempo
+    final_tempo = known_tempo if known_tempo is not None else measured_tempo
     if (
         known_tempo is not None
         and measured_tempo is not None
@@ -243,7 +243,7 @@ def measure_grid(
             f"measures {measured_tempo} — using yours; drop tempoBpm from "
             "grid= to use the measured value",
         )
-    final_origin = known_origin or measured_origin
+    final_origin = known_origin if known_origin is not None else measured_origin
     if known_origin is None and measured_origin is not None:
         flags += (
             f"origin-estimated: first beat of the music region "
@@ -267,11 +267,19 @@ def measure_grid(
         confidence = UNMEASURED_CONFIDENCE
     else:
         confidence = (
-            KNOWN_ORIGIN_CONFIDENCE if known_origin else DFLT_MEASURED_CONFIDENCE
+            KNOWN_ORIGIN_CONFIDENCE
+            if known_origin is not None
+            else DFLT_MEASURED_CONFIDENCE
         )
 
     spu = seconds_per_unit(grid)
-    if total_units and total_units > 0 and spu and final_origin and region:
+    if (
+        total_units
+        and total_units > 0
+        and spu
+        and final_origin is not None
+        and region is not None
+    ):
         routine_s = total_units * spu
         region_s = region.end - float(Fraction(final_origin))
         evidence.update(routine_s=round(routine_s, 2), region_s=round(region_s, 2))

{paces-0.0.20 → paces-0.0.21}/paces/model.py

@@ -345,7 +345,11 @@ def resolve(doc: StepDocument) -> dict:
     """
     grid = doc.metric
     spu = seconds_per_unit(grid) if grid is not None else None
-    origin = float(Fraction(grid.origin)) if grid and grid.origin else None
+    origin = (
+        float(Fraction(grid.origin))
+        if grid is not None and grid.origin is not None
+        else None
+    )
 
     def _wall(offset_units: float) -> float | None:
         if spu is None or origin is None:

{paces-0.0.20 → paces-0.0.21}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "paces"
-version = "0.0.20"
+version = "0.0.21"
 description = "Turn instructional media into structured, interactive learning material"
 readme = "README.md"
 license = "MIT"

{paces-0.0.20 → paces-0.0.21}/tests/test_segment.py

@@ -199,3 +199,35 @@ def test_place_on_grid_names_what_is_missing(steps, grid, match):
 
     with pytest.raises(ValueError, match=match):
         place_on_grid(steps, grid)
+
+
+@pytest.mark.parametrize("origin, start", [("0", 0.0), ("0.0", 0.0), ("51.2", 51.2)])
+def test_resolve_computes_wall_clock_from_any_origin_including_zero(origin, start):
+    """An origin of 0 is a real origin, not a missing one (#46)."""
+    from paces.model import resolve
+
+    doc = to_document(
+        segment(steps=[("a", 1)], grid={**GRID, "origin": origin}),
+        doc_id="d",
+        title="t",
+    )
+    assert resolve(doc)["steps"][0]["start_s"] == pytest.approx(start)
+
+
+def test_resolve_without_an_origin_has_no_wall_clock():
+    from paces.model import resolve
+
+    doc = to_document(segment(steps=[("a", 1)], grid=GRID), doc_id="d", title="t")
+    doc = doc.model_copy(
+        update={"metric": doc.metric.model_copy(update={"origin": None})}
+    )
+    assert resolve(doc)["steps"][0]["start_s"] is None
+
+
+def test_resolve_treats_a_numeric_zero_origin_as_an_origin():
+    """A grid updated in place skips wire validation; 0 must still be 0 (#46)."""
+    from paces.model import resolve
+
+    doc = to_document(segment(steps=[("a", 1)], grid=GRID), doc_id="d", title="t")
+    doc = doc.model_copy(update={"metric": doc.metric.model_copy(update={"origin": 0})})
+    assert resolve(doc)["steps"][0]["start_s"] == 0.0