ContinuumIO / whitebox-geospatial-analysis-tools

An open-source GIS and remote sensing package
http://www.uoguelph.ca/~hydrogeo/Whitebox/
7 stars 7 forks source link

Deduplicate code / cleanup in whitebox_tools/whitebox_*.py #15

Open PeterDSteinberg opened 7 years ago

PeterDSteinberg commented 7 years ago

The xarray wrapping of Whitebox tools within this fork worked around existing logic for subprocess management (i.e. adding to existing code where it may be better to combine / simplify). If I recall correctly, some of the code in whitebox_tools/whitebox*.py could be simplified. Review that code and if anything can be removed or clarified, specifically with respect to subprocesses and piping/printing the outputs of subprocesses.