search

geodesymiami / rsmas_insar

RSMAS InSAR code
https://rsmas-insar.readthedocs.io/
GNU General Public License v3.0
62 stars 23 forks source link

Different Command Line Argument Parsing Schemes are Confusing #1

Closed Ovec8hkin closed 6 years ago

Ovec8hkin commented 6 years ago

download_asf_rsmas.py and download_ssara_rsmas.py currently use a modified command line argument parsing scheme that does not rely on the python argparse library. Compared with the rest of the codebase which uses argparse, this can be confusing to edit/update, especially in the event that new command line argument need to be passed to the function.

Should be updated so as to use argparse and position arguments rather than sys.argv[1]

Ovec8hkin commented 6 years ago

Completed with commit e82f764