Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start grabbing some French data #21

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Start grabbing some French data #21

wants to merge 1 commit into from

Conversation

dabreegster
Copy link
Collaborator

A start to #13. I've gone for the absolute simplest shell script-like approach, producing a GeoJSON file and a CSV file with a matching zone ID column. Thanks to @XioNoX for finding the data!

# Produces working_dir/zones.geojson in WGS84, with only one property, "iris_code"
def get_geometries(working_dir):
# 2023 IRIS areas for all of France, from https://geoservices.ign.fr/irisge
# TODO Probably need to use 2021, to match up with the census spreadsheet (https://wxs.ign.fr/0aezh2n2288zcsb0gyx38xfs/telechargement/prepackage/IRIS-GE-TERRITOIRES-PACK_2019$IRIS-GE_2-0__SHP_LAMB93_FXX_2019-01-01/file/IRIS-GE_2-0__SHP_LAMB93_FXX_2019-01-01.7z)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm on slow internet right now, but can switch later. Probably some paths below need to change!

@XioNoX
Copy link

XioNoX commented Oct 24, 2023

Can I maybe help with this ? Not sure what the end result is supposed to be.

@dabreegster
Copy link
Collaborator Author

Thanks for the offer! I think we're OK for the moment; Stuart and Andy are experimenting with using https://dagster.io/ to organize code in this repo. Once some of the dust settles there, we'll pick up the work for France

@andrewphilipsmith
Copy link
Collaborator

There is now some guidance on how to add a new country https://popgetter.readthedocs.io/en/latest/new_country/

However this might change a bit more in the future. For now, I am marking this PR as draft.

@andrewphilipsmith andrewphilipsmith marked this pull request as draft August 8, 2024 10:34
@dabreegster
Copy link
Collaborator Author

@XioNoX, if you'd like to pick this up, let us know! It might be lots of work if you haven't used Dagster before though

@XioNoX
Copy link

XioNoX commented Aug 13, 2024

I'd be happy to, but after reviewing the doc and the existing countries, it seems like out of my skills level.
I can however provide guidance and answer questions on french specific data, or if one of you have a bit of time to walk me through the process that would be nice !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog:
Development

Successfully merging this pull request may close these issues.

3 participants