search

opengeos / open-buildings

Tools for working with open building datasets
https://opengeos.github.io/open-buildings
Other
124 stars 17 forks source link

Structure of the package #46

Open felix-schott opened 11 months ago

felix-schott commented 11 months ago

Description

Two things that I'm unclear about:

  1. Not quite sure what the purpose of common.py is - can this be deleted?
  2. I would also recommend to move utility functions to a utils.py file and further put all CLI handlers in the cli.py file, with a clear separation of CLI handler (command line interface) and library function (python interface). E.g. not quite clear sure why download_buildings also contains CLI handlers.