adjtomo / pysep

Seismogram Extraction and Processing: Seismic data retrieval and record sections
https://pysep.readthedocs.io
MIT License
28 stars 15 forks source link

small annoyances. #107

Closed aileen-fisher closed 1 year ago

aileen-fisher commented 1 year ago

In your documentation: https://adjtomo.github.io/pysep/pysep.html,

pysep -p MTUQ2022_workshop -e 2017-09-03T033001_NORTH_KOREA.yaml is wrong. Should be mtuq_workshop_2022.

Also, 'pysep -W' will overwrite another saved config file of the same name without asking. It should ask if overwrite is ok or just add something to the end of the new file name.

bch0w commented 1 year ago

Hi @aileen-fisher, thanks for pointing those out!

I've pushed a few changes to the latest devel branch that fixes the docs (https://github.com/adjtomo/pysep/blob/devel/docs/pysep.rst#pysep-via-command-line) and does not let users write over existing files with -W unless they use the -o flag as well.

These will be included in the next official version release.

bch0w commented 1 year ago

Closing as this has been addressed in the latest version. Please re-open if you're still encountering these issues!