Dev jojomale #83
Merged
Dev jojomale #83
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 14 commits
November 6, 2024 10:33
Replace `or` by `|` in union types.
- Introduce `waveform.Local_store_client` to access existing sds data base - introduce additional input parameters `stationxml_file`, `sds_dir`, `sds_fmtstr` to allow for datafiles to be in different locations on the computer and to have different filenames than the standard sds-pattern. - `Correlator` can be called without providing an externally defined client. In this case, it creates a Local_store_client from the parameter file.
original requirements caused UserWarnings with h5py and mpi4py/openmpi upon import, although the execution of seismic didn't seem to suffer. The updated package versions were obtained by uninstalling and reinstalling the respective packages without version restriction in the same conda environment.
commit 17593c649bcdc7b65af19336564eee4c4815d26c
Author: jojomale <jlehr@gfz-potsdam.de>
Date: Wed Dec 4 15:10:06 2024 +0100
Update tests to new Correlator and config parameters.
commit f4f882a01480512a89d690770e68db80fdac6ea5
Author: jojomale <jlehr@gfz-potsdam.de>
Date: Wed Dec 4 14:56:47 2024 +0100
Update documentation to new Correlator and config parameters.
commit 695068ab0551ba3561f3d2375843937f8718332d
Author: jojomale <jlehr@gfz-potsdam.de>
Date: Wed Dec 4 14:11:54 2024 +0100
Change order of input params in Correlator.
With the new Local_Store_Client, there is no need to have a download
client at all. Thus it is more intuitive to
have the store_client as an optional
argument. The local store client will
be initiated with the Correlator from
the configs.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Main change is the introduction of a more flexible local store client (basically an sds-client), new parameters for waveform and station-file directories, and order of arguments to Correlator. The new settings allow to use a local database which is not in the default directory structure of seismic. Also the filename format can now be set, thus file names do not strictly have to be standard SDS-format. This should make it easier for users to use their existing data base.
Moreover, the Correlator now takes only the config-parameters as obligatory argument. The store-client is optional.
Users might need to change existing scripts when updating.