You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Consolidation of the three rolling sweep issues, filed on espg's direction; #73, #88, and #97 are closed in favor of this issue. Every item from all three was re-verified against current main (post PR #106 merge). Resolved items are listed once for the record; everything still open is grouped by what it needs next, with a priority tag per item.
Resolved since the sweeps (no action — for the record)
Group A — small-fixes, bundleable into ONE claude/small-fixes-<date> PR
(each has a standing recommendation; a 👍 on this issue + small-fix labels activate the bundle)
[low]from_geometry(..., normalize=False) is silently a no-op for LineString/MultiLineString — raise ValueError in the linear branch, symmetric with how moc/tolerance/max_cells are rejected there (mortie/geometry.py ~262) + test. (from Follow up items for the week of July 4, 2026 #97 item 7)
[medium] Dissolve winding-sign ambiguity near/over a hemisphere — add the runtime guard that raises when |Σ areas| approaches 2π (fail loud instead of silently inverting exterior/hole on polar-scale covers) + a near-hemisphere test; the comment-only status quo stays risky. Recommended (a) from Follow up items for the week of July 4, 2026 #97 item 5; the full winding-free classifier remains a possible follow-up if polar dissolves become a real workload.
[high, awaiting merge]res2display() order cap + km/m/cm ladder → draft PR Fix res2display() order cap and add km/m/cm unit ladder #102 (Closes #99), all phases complete, CI green, waiting on your review since July 5. Merging it discharges the last high-priority sweep item.
🤖 from Claude
Consolidation of the three rolling sweep issues, filed on espg's direction; #73, #88, and #97 are closed in favor of this issue. Every item from all three was re-verified against current
main(post PR #106 merge). Resolved items are listed once for the record; everything still open is grouped by what it needs next, with a priority tag per item.Resolved since the sweeps (no action — for the record)
on_minor_arcFP sensitivity → implemented in PR SoS-harden on_minor_arc in robust_crossing #87, then superseded by the uniform predicate in PR Uniform symbolic crossing predicate: fix base-cell-boundary mis-fill #106; on_minor_arc FP/SoS hardening #78 carries a close recommendation.mort2normFORCE_PYTHON remnant + stale FORCE_PYTHON benchmark scripts → removed.from_latlonpoint-encode (points=True) → landed via issue numpy-level point-kind geo2mort: geo2mort(lat, lon, order=29, points=True) #96 (phases 1–5 onmain).__from_arrow__hook → wired (mortie/morton_index.py).moc_to_orderguards up front (issue pre-emptive flat-cover size guard #80 / PR small fixes 2026-06-27: densify size guard (#80) + morton_polygon determinism tests (#83) #85).smallvecforNode.relevant→ signed off and in (src_rust/src/coverage.rs).arro3-no-pyarrowjob added (issue arro3-core CI leg for the pyarrow-free consumer was designed but never wired up #98) and green on every PR since.__repr__,split_base_cellsordering + wrapper decisions → resolved per Follow up items for the week of July 4, 2026 #97's re-verification.mktemp+mvvssed -iportability sign-off → moot (shipped deliberately; noting closes it).Group A — small-fixes, bundleable into ONE
claude/small-fixes-<date>PR(each has a standing recommendation; a 👍 on this issue +
small-fixlabels activate the bundle)from_geometry(..., normalize=False)is silently a no-op for LineString/MultiLineString — raiseValueErrorin the linear branch, symmetric with howmoc/tolerance/max_cellsare rejected there (mortie/geometry.py~262) + test. (from Follow up items for the week of July 4, 2026 #97 item 7)useless_conversionfalse positives (31 at the Follow up items for the week of July 4, 2026 #97 count, ~38 now) — add#[allow(clippy::useless_conversion)]at the crate root with a one-line comment that the pyo3/numpy?conversion is load-bearing. Has bounced through #34 §D cleanup + fmt/clippy sweep #57 → Follow up items for the week of June 23, 2026 #73 → Follow up items for the week of June 27, 2026 #88 → Follow up items for the week of July 4, 2026 #97 undecided; this is the recommended (a). (from Follow up items for the week of June 23, 2026 #73 item 8)|Σ areas|approaches 2π (fail loud instead of silently inverting exterior/hole on polar-scale covers) + a near-hemisphere test; the comment-only status quo stays risky. Recommended (a) from Follow up items for the week of July 4, 2026 #97 item 5; the full winding-free classifier remains a possible follow-up if polar dissolves become a real workload.Group B — needs an espg decision before any code
//floor-division Morton coarsening operator — fork unchanged since Follow up items for the week of July 4, 2026 #97: (a) ship__floordiv__alone now, or (b) design the full NEP-42 operator grammar in onediscussso a partial grammar isn't frozen into 1.x. Standing recommendation: (b), given the 1.x freeze contract. (from Follow up items for the week of June 23, 2026 #73/Follow up items for the week of June 27, 2026 #88/Follow up items for the week of July 4, 2026 #97)moc_to_ordersize-guard pair from PR small fixes 2026-06-27: densify size guard (#80) + morton_polygon determinism tests (#83) #85 — (i) keepValueError(recommended: it's a caller-controlled argument condition, not a failed allocation) vsMemoryError; (ii) keep the hard raise pinned to the1<<20warn line vs a separate higher ceiling (recommended: separate, higher). One-pass answers; then it's a trivial diff that can ride the Group-A bundle. (from Follow up items for the week of July 4, 2026 #97 item 8)prefix_trie.morton_polygontie-break non-determinism (equal-efficiency cells since the #34 §D cleanup + fmt/clippy sweep #57 max-heap) — document as implementation-defined, or pin with a determinism test. Never individually tracked; cheap either way once you pick. (from Follow up items for the week of June 23, 2026 #73 item 9)Group C — in flight, no action here
res2display()order cap + km/m/cm ladder → draft PR Fix res2display() order cap and add km/m/cm unit ladder #102 (Closes #99), all phases complete, CI green,waitingon your review since July 5. Merging it discharges the last high-priority sweep item.Suggested sequencing
small-fixgo-ahead on Group A → one bundled PR (Closesper item).//grammar, notebooks) as their owndiscuss/planwhen you want them.Refs #73, #88, #97 (closed into this issue).