search

mapbox / make-surface

Vector surfaces creation routines
MIT License
89 stars 18 forks source link

More CLI harmony. #64

Closed sgillies closed 9 years ago

sgillies commented 9 years ago

I'm suggesting a couple changes in the interest of harmony between makesurface and rasterio and a more common CLI feel. Let's have our common options be as uniform as we can make them, it makes them easier to use.

Vectorize

  1. Change the --band option to --bidx as in rio-stack etc. I like -b as the short option name and will make sure that's what rio uses.
  2. Change the short option name for --nodata to -n as in GDAL programs. This means moving the one for --nibble to -ni, but I feel like --nodata is the more commonly used option across all our scripts.

    Triangulate

  3. Change --bbox to --bounds as in rio-merge and make it a 4 float tuple (I've followed up in the function that uses it).
  4. Change --tile to a 3 float tuple (ditto).
sgillies commented 9 years ago

@dnomadb the make-surface CLI is looking good. The stdout or --output option is just right.

dnomadb commented 9 years ago

Going to merge this into https://github.com/mapbox/make-surface/tree/point-sampler first