Repository navigation
Numedalslågen features clip ruled expected at 68 % (hotspots.toml); ROADMAP rows 35, 36, 26 and their Status lines to merged - #233
Merged
Conversation
…OADMAP row 35 merged, by the main session Ola, 2026-10-12, asked whether to rule Numedalslagen's features clip (6.0 s of 8.8 s, 68 %, docs/benchmarks/2026-10-11/cleanup-speed/README.md) expected and leave the fix to the cached-extract idea: "Rule it expected at 6 s, cached extract is the fix". The hotspots.toml entry carries the words verbatim, so the quick check stays quiet until the share grows 10 points. Lagan's 43.6 % is within 10 points of its 40.7 % ruling and gets no new entry. ROADMAP row 35: built -> merged (#229); its stale "next the push" phrase replaced by the ruling. Increment 35's section 11 gets the ruling in one sentence after the default-taken line. Checked: tests/python/test_bench_quick.py -k shipped, 2 passed (hotspots.toml loads, every entry has a ruling). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…ain session Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…main session Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…ings and 36's Status fixed, by the main session Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… to the reviewer's wording, by the main session Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Records only; no production code (
count_loc.py: 0).docs/benchmarks/quick/hotspots.toml: a new[[hotspot]]entry for Numedalslågenfeatures clipat 68.0 % (6.000 s of 8.829 s,docs/benchmarks/2026-10-11/cleanup-speed/README.md), with Ola's ruling of 2026-10-12 word for word: "Rule it expected at 6 s, cached extract is the fix". The quick check stays quiet until the share grows ten points. Lagan's 43.6 % is within ten points of its 40.7 % ruling and gets no entry.ROADMAP.md: row 35 built -> merged (Increment 35: the land-cover clean-up step made faster, output byte-identical #229) with the ruling; row 36 built -> merged (Increment 36: GeoPackage output (--out X.gpkg) over sqlite3, QGIS-ready #231); row 26 "26a built" -> "26a merged (Increment 26a: MapBiomas catalogue entries and per-year land-cover class windows #232)".Review: two pre-push rounds recorded at the end of
docs/increments/35-cleanup-speed.md, both rounds' wordings applied;check_citations.pyclean; the shipped-hotspots tests pass.🤖 Generated with Claude Code