NeoGeographyToolkit / StereoPipeline

The NASA Ames Stereo Pipeline is a suite of automated geodesy & stereogrammetry tools designed for processing planetary imagery captured from orbiting and landed robotic explorers on other planets.
Apache License 2.0
492 stars 173 forks source link

mapproject: switch gdalbuildvrt command to subprocess.call() #299

Closed mazhe closed 4 years ago

mazhe commented 4 years ago

Hi, I had an issue with mapproject and some big HiRISE imagery...

os.system() can be a bit limited, such as when dealing with lots of tiles and fail silently due to subshell issues. subprocess.call() (with shell=False) is better at this and fails with more understandable messages if needed.

Here's a proposed fix, would you consider integrating it?

Thanks for all your work!

oleg-alexandrov commented 4 years ago

Thank you. :)