Skip to content

Commit 781a3ec

Browse files
asinghvi17claude
andcommitted
Record validation-gate results in the PR #428 response plan
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 6ad05ec commit 781a3ec

1 file changed

Lines changed: 18 additions & 0 deletions

File tree

docs/plans/2026-07-04-pr428-review-response.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -304,3 +304,21 @@ Executed as planned, one commit per theme:
304304
Gates: prepared + polygon-clipping suites after each theme (9828
305305
assertions across the 3-implementation matrix), full `Pkg.test()` and the
306306
Natural Earth benchmark at the end.
307+
308+
All gates passed:
309+
310+
- Full `Pkg.test()` green (Prepared geometry 330, Polygon Clipping 12 691,
311+
everything else unchanged).
312+
- Clipping micro-benchmark (1280-vert holed donut × 1024-vert blob,
313+
`intersection_points` on whole polygons): NestedLoop 9 588 µs;
314+
DoubleNaturalTree plain 40.9 µs; DoubleNaturalTree prepared **15.0 µs**;
315+
Auto on prepared inputs 14.3 µs — the Theme B change makes
316+
`AutoAccelerator` hit the dual-reuse path, matching the explicit dual
317+
accelerator. Prepared:plain-tree ratio ≈ 2.7×, same shape as the
318+
recorded 22.8 µs vs 70 µs (absolute numbers machine-load dependent).
319+
Boolean intersection: prepared 446 µs ≤ plain 465 µs; results verified
320+
equal against NestedLoop ground truth.
321+
- Natural Earth 110m PIP: NaturalIndex 4.6× whole-dataset vs plain
322+
(52 ns/q prepared vs 239 ns/q plain), 21× on the 100–1k bucket,
323+
break-even ≈ 12 queries/polygon; all backends agree on all 288 polygons;
324+
no backend slower than plain. In line with the recorded 5.0× / 30×.

0 commit comments

Comments
 (0)