aiddata / gcdf-geospatial-data

Repository for AidData's Geospatial Global Chinese Development Finance Dataset (GeoGCDF)
https://aiddata.org/china
Other
31 stars 8 forks source link

Refactor / restructure #5

Open sgoodm opened 3 years ago

sgoodm commented 3 years ago

Entire codebase is currently in a single Python file that is essentially just a final proof-of-concept / prototype. We would likely benefit (even if only in terms of readability) by breaking some of the functions out into a class in a separate file.

This would be a good opportunity to make sure code is well organized and documented with comments/docstrings.

sgoodm commented 3 years ago

Eventually we may want to create a dedicated package for extracting spatial features from OSM links so that this can be applied to other projects and used by other folks.

sgoodm commented 3 years ago

I've done a fair amount of cleaning up, documenting code, etc. but may still want to explore viability of classes and refactoring some components (and creating a dedicated package)