EC-FR: the 2018 campaign, from EuroCrops - #288
Merged
Merged
Conversation
ivorbosloper
added a commit
that referenced
this pull request
Sep 15, 2026
ivorbosloper
added a commit
that referenced
this pull request
Sep 15, 2026
IGN never released an RPG archive for 2018; the downloads run 2017 and 2019 onwards. EuroCrops publishes it with the HCAT columns already resolved, so the collection is theirs rather than a variant of fr. 9,517,878 parcels, ids unique. SURF_PARC is rounded to 0.01 ha, so 4,913 parcels under 50 m2 read as zero and get their area from the geometry. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DVx9uQV2QPM8ecPAY3ZjXG
Zenodo answers with two Content-Type headers and the aiohttp client behind the base class rejects that with a 400, so no ec_* converter can download on a cold cache. The failed attempt also left a zero-byte file behind, which the next run would have taken for a cached archive.
ivorbosloper
force-pushed
the
split/ec_fr
branch
2 times, most recently
from
September 15, 2026 18:07
59e4ed0 to
2ce50ce
Compare
ivorbosloper
added a commit
that referenced
this pull request
Sep 15, 2026
Splitting early only makes that possible; it does not do it. Nine parcels are multi-part, so thirteen rows shared an id.
ivorbosloper
added a commit
that referenced
this pull request
Sep 15, 2026
m-mohr
reviewed
Sep 15, 2026
Zenodo answers with two Content-Type headers. aiohttp rejected that until 3.13.5, which skips the RFC 9110 singleton check in lax mode -- the default for responses (aio-libs/aiohttp#12302). The lock held 3.13.3, so no ec_* converter could download on a cold cache. The requests fallback added for it goes again.
m-mohr
approved these changes
Sep 17, 2026
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
2 participants
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.
Based on #286, which adds the
split_multipartthis uses.IGN never released an RPG archive for 2018 — the downloads run 2017 and 2019 onwards — so
frhas a gap there. EuroCrops publishes that campaign with its HCAT columns already resolved, so this is a collection of its own rather than a variant offr.ID_PARCELunique. 13 are multi-part and are split before the id is checked.SURF_PARCis rounded to 0.01 ha, so 4,913 parcels under 50 m² read as zero and take their area from the geometry.Content-Typeheaders, which the aiohttp client behind the base class rejects with a 400. Noec_*converter could download on a cold cache.Closes fieldsoftheworld/harmonized-field-data-catalog#13 once published.