search

usgs / groundmotion-processing

Parsing and processing ground motion data
Other
54 stars 41 forks source link

Add overwrite option for generate_report #1046

Closed kallstadt-usgs closed 1 year ago

kallstadt-usgs commented 1 year ago

Is your feature request related to a problem? Please describe. If I want to remake plots and reports after changing some processing options, for example, if the /plots folder already exists, the code crashes (FileExistsError: [Errno 17] File exists: 'somefolder/data/us6000hynr/plots'). Right now the fix is to manually delete the plots folder before running.

Describe the solution you'd like Add an overwrite option flag (-o) like other commands have to generate_report

baagaard-usgs commented 1 year ago

Do you want an overwrite flag or should the default be to just overwrite?

emthompson-usgs commented 1 year ago

That's a good point. I don't see a downside to having the default be to overwrite.