Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

run_id #6

Merged
merged 10 commits into from
Jul 18, 2022
Merged

run_id #6

merged 10 commits into from
Jul 18, 2022

Conversation

krisvanneste
Copy link
Collaborator

This feature branch adds a --run-id command-line argument to source_spec. The run_id is used to name the output folder and is also added in a dedicated field in the database. This allows running source_spec with different configurations for the same event without overwriting the corresponding results.

@claudiodsf
Copy link
Member

Hi kris, thanks for the pull request.

I'm adding a few extra commits and I'll let you know when I'm done.

@claudiodsf claudiodsf force-pushed the run_id branch 2 times, most recently from 2dff555 to e35ff4d Compare July 18, 2022 06:30
claudiodsf added a commit to krisvanneste/sourcespec that referenced this pull request Jul 18, 2022
@claudiodsf
Copy link
Member

Hi @krisvanneste,
that's good for me.

Let me know if it's ok for you as well, so that I'll merge.

@krisvanneste
Copy link
Collaborator Author

Claudio,
Thanks for your improvements!
This branch works OK for me.

Kris

@claudiodsf claudiodsf merged commit 6c934ac into SeismicSource:master Jul 18, 2022
@claudiodsf
Copy link
Member

Great! Merged! Thanks again

claudiodsf added a commit to krisvanneste/sourcespec that referenced this pull request Jul 18, 2022
* Added `--run_id` command-line argument in `_update_parser` function.

* Add optional `run_id` to new `outdir` name in `move_outdir` function.

* Add `runid` field to station and event tables in `_write_db` function.

* Write `run_id` (if not empty) to parfile

* Write `run_id` (if not empty) to HTML report

* Changelog for PR SeismicSource#6

* Add Kris to contributors ;-)

Co-authored-by: Claudio Satriano <satriano@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants