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

Make input geometry configurable #85

Open
wants to merge 3 commits into
base: staging
Choose a base branch
from

Conversation

sjockers
Copy link
Collaborator

Make it easier to use this project for regions other than Berlin by adding .env parameters for configuring the source shapefile for the buffer:

  • Path to input shapefile
  • Distance to use for creating the buffer geometry

Defaults to previous settings (Berlin shapefile, buffer distance of 2000).

[This is a local duplicate of #82 in order to properly run checks, with access to secrets]

Shapely.cascaded_union is deprecated, but as this change may have unforseen effects, it should really be done in a separate PR.
@ff6347
Copy link
Collaborator

ff6347 commented Apr 29, 2023

Very nice thank you. I'll give it a spin next week.

I'd love to have a workflow that runs this script as well. Would be a great addition to this PR. (Wink wink nudge nudge) if your not able to work on this I will delay the approval until we have a working action

@ff6347
Copy link
Collaborator

ff6347 commented Apr 29, 2023

Delayed until workflow exists #86

@ff6347 ff6347 self-assigned this Apr 29, 2023
@Jaszkowic
Copy link
Collaborator

thanks @sjockers , in the meantime we have refactored the harvester quite a lot, but your proposal is still useful! in case you can find the time feel free to merge staging and resolve the conflicts 👍🏼

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

Successfully merging this pull request may close these issues.

3 participants