Fill no-one's-land gaps between merged sub-regions (v0.17.0) #32

2026-10-13T17:45:23+02:00

kamilkrzywda commented

2026-10-13 17:45:23 +02:00

Part of #16

Fixes the unfilled "no-one's-land" holes left when the v0.14 coarsening merged sub-regions into bigger ones — the real problem behind the Moselle (FR) / Kočevje (SI) / Kavadartsi (MK) / Ağcabədi (AZ) holes.

Approach: new scripts/fill-gaps.mjs (npm fill:regions) uses each country's authoritative landmass (public/countries.geo.json, id=ISO_A3) as the reference. For every country it computes gaps = country_landmass − union(that country's regions) — exactly the no-one's-land — and assigns each gap to the nearest region of that country (each gap → one region, so regions never overlap). Sub-25 km² border-precision slivers are skipped; cross-border phantom overhang is clipped away.

Result:

All 1,796 regions are now solid / fully connected (verified: every named region and, globally, 1,793/1,796 collapse to a single connected polygon when their parts are unioned; 0 still fragmented). The added gap pieces touch the existing parts, so the holes are genuinely filled — not new floating fragments.
Total no-one's-land dropped ~3.9M → 63,889 km² (98.4%); the residual is coastline digitization fuzz plus the deliberately-kept KZ-KZY Aral-basin lake (it's water, excluded from the landmass, so it correctly stays a hole).
1,796 regions / all 177 countries / every id+name+parent unchanged; all 83 islands kept; no overlaps; idempotent.
File grew 2.29 → 3.40 MB (the touching pieces are stored as separate parts; a follow-up can dissolve each region to a single polygon to shrink it if the map feels heavy).

Map/LLM contract untouched — this only rewrites public/regions.geo.json geometry. Adds a dataset test (no significant no-one's-land per country + KZ-KZY hole present).

Part of #16

Part of #16 Fixes the unfilled "no-one's-land" holes left when the v0.14 coarsening merged sub-regions into bigger ones — the real problem behind the Moselle (FR) / Kočevje (SI) / Kavadartsi (MK) / Ağcabədi (AZ) holes. **Approach:** new `scripts/fill-gaps.mjs` (npm `fill:regions`) uses each country's authoritative landmass (`public/countries.geo.json`, `id`=ISO_A3) as the reference. For every country it computes `gaps = country_landmass − union(that country's regions)` — exactly the no-one's-land — and assigns each gap to the nearest region of that country (each gap → one region, so regions never overlap). Sub-25 km² border-precision slivers are skipped; cross-border phantom overhang is clipped away. **Result:** - All **1,796 regions are now solid / fully connected** (verified: every named region and, globally, 1,793/1,796 collapse to a single connected polygon when their parts are unioned; 0 still fragmented). The added gap pieces *touch* the existing parts, so the holes are genuinely filled — not new floating fragments. - Total no-one's-land dropped ~3.9M → 63,889 km² (98.4%); the residual is coastline digitization fuzz plus the deliberately-kept **KZ-KZY Aral-basin lake** (it's water, excluded from the landmass, so it correctly stays a hole). - **1,796 regions / all 177 countries / every `id`+`name`+`parent` unchanged**; all 83 islands kept; no overlaps; idempotent. - File grew 2.29 → 3.40 MB (the touching pieces are stored as separate parts; a follow-up can dissolve each region to a single polygon to shrink it if the map feels heavy). Map/LLM contract untouched — this only rewrites `public/regions.geo.json` geometry. Adds a dataset test (no significant no-one's-land per country + KZ-KZY hole present). Part of #16

kamilkrzywda added 1 commit 2026-10-13 17:45:23 +02:00

Fill no-one's-land gaps between merged sub-regions (v0.17.0)

CI / build + unit + e2e (pull_request) Successful in 2m19s

Details

b1081eefde

Part of #16

kamilkrzywda scheduled this pull request to auto merge when all checks succeed 2026-10-13 17:48:16 +02:00

kamilkrzywda merged commit acb4f0800f into master

2026-10-13 17:48:18 +02:00

kamilkrzywda deleted branch feat/fill-gaps

2026-10-13 17:48:18 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: OpenCode/History#32