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.

Sign up for GitHub

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

Jump to bottom

add layers for fire stations and hospitals #442

Merged
merged 16 commits into from
Dec 6, 2022
Merged

add layers for fire stations and hospitals #442

merged 16 commits into from
Dec 6, 2022

Conversation

Gigaszi
Copy link
Contributor

@Gigaszi Gigaszi commented Oct 3, 2022 •
edited

Description

Add layers fire_station_count and hospital_count.

Corresponding issue

Closes #409

New or changed dependencies

None

Checklist

  • I have updated my branch to main (e.g. through git rebase main)
  • My code follows the style guide and was checked with pre-commit before committing
  • I have commented my code
  • I have added sufficient unit and integration tests
  • I have updated the CHANGELOG.md

matthiasschaub
matthiasschaub reviewed Oct 11, 2022
View reviewed changes
Copy link
Collaborator

@matthiasschaub matthiasschaub left a comment

Choose a reason for hiding this comment

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

Which indicators could use those layer definitions. Sould we not update the indicator/layer combination list so that those layer definitions can be used?

matthiasschaub
matthiasschaub previously approved these changes Nov 29, 2022
View reviewed changes
@matthiasschaub matthiasschaub added this to the Release 0.14.0 milestone Dec 6, 2022
@matthiasschaub matthiasschaub merged commit af01e61 into main Dec 6, 2022
@matthiasschaub matthiasschaub deleted the emergency_layers branch December 6, 2022 15:42
@matthiasschaub matthiasschaub mentioned this pull request Feb 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthiasschaub matthiasschaub matthiasschaub approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 0.14.0
Development

Successfully merging this pull request may close these issues.

Add new layer definition: Number of hospitals
2 participants
@Gigaszi @matthiasschaub