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

[DEV] refactor osm_maps_functions.py like geofabrik.py - using a interface and two implementing classes #194

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

treee111
Copy link
Owner

@treee111 treee111 commented Apr 3, 2023

This PR…

  • there is now a Country and XY OsmData class in the same manner as geofabrik.py

Considerations and implementations

aligned with https://realpython.com/python-interface/

How to test

  1. test like normally
  2. run unittests

Pull Request Checklist

@treee111 treee111 added this to the v4.0.0 milestone Apr 3, 2023
@treee111 treee111 changed the title [DEV] refactor osm_maps_functions.py like geofabrik.py - Interface [DEV] refactor osm_maps_functions.py like geofabrik.py - using a interface and two implementing classes Apr 3, 2023
@treee111 treee111 merged commit aa5b541 into release-v4.0.0 Apr 3, 2023
@treee111 treee111 deleted the refactor-osm-data branch April 3, 2023 08:03
treee111 added a commit that referenced this pull request May 5, 2023
…rface and two implementing classes (#194)

refactor osm_maps_functions.py like geofabrik.py
alfh pushed a commit to alfh/wahooMapsCreator that referenced this pull request Nov 4, 2023
…rface and two implementing classes (treee111#194)

refactor osm_maps_functions.py like geofabrik.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant