Skip to content

Replace data.cquest.org downloads with carte.gouv.fr downloads#291

Merged
FlxPo merged 3 commits intomainfrom
fix-cquest-download
Mar 30, 2026
Merged

Replace data.cquest.org downloads with carte.gouv.fr downloads#291
FlxPo merged 3 commits intomainfrom
fix-cquest-download

Conversation

@FlxPo
Copy link
Copy Markdown
Contributor

@FlxPo FlxPo commented Mar 30, 2026

Motivation

data.cquest.org downloads for the ADMIN EXPRESS CARTO dataset sometimes fail (as they do today).
See #290 for a bug report.

Changes

This PR replaces the url of the dataset on data.cquest.org by the url of the same dataset on cartes.gouv.fr. This server is less likely to be down because cartes.gouv.fr is an official platform backed by the french government. The url might not be stable long term so we'll may have to cache the dataset on data.gouv.fr at some point.

AI-assisted contribution

Select one:

  • No AI assistance

Checklist

  • I have reviewed all code in this PR
  • I understand the code and can maintain it
  • I added or ran appropriate tests/checks for the changed behavior

@FlxPo
Copy link
Copy Markdown
Contributor Author

FlxPo commented Mar 30, 2026

The new url works but the dataset folder structure changed, I have to check locally. Also the download speed is pretty slow... We should probably switch to distributing our own versioned extract of this dataset (in parquet format), to speed things up.

@FlxPo FlxPo added the bug Something isn't working label Mar 30, 2026
@FlxPo FlxPo self-assigned this Mar 30, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.06%. Comparing base (3d1c109) to head (2f5ff0d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #291   +/-   ##
=======================================
  Coverage   59.06%   59.06%           
=======================================
  Files         129      129           
  Lines        6738     6738           
=======================================
  Hits         3980     3980           
  Misses       2758     2758           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@FlxPo FlxPo merged commit 79769a7 into main Mar 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant