-
Notifications
You must be signed in to change notification settings - Fork 11
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
add from_open_street_map() Methods to Model Class #350
Comments
This is a good idea. I would just note a few things: One thing I would also note is that, if we add a method for this, we should write out |
Ah my thinking was to use the following information from Open Street Map:
The idea was that OSM is a strongly typed semantic map (more so than GeoJson which can hold the same information but name of the key values is likely to vary more). Agreed about using Is this something you reckon I can start working on or should I put it on hold like the |
Open Street Map is a strongly formatted and rich data format to host building information. I reckon adding this as a method might be a bit of a stretch as it would involve writing an XML parser for OSM etc... But might as well drop the issue here in case it gains traction.
The text was updated successfully, but these errors were encountered: