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

Improve phase association #111

Merged
merged 10 commits into from
Jun 24, 2022
Merged

Improve phase association #111

merged 10 commits into from
Jun 24, 2022

Conversation

damb
Copy link
Collaborator

@damb damb commented Jun 24, 2022

Features and Changes:

  • Improve phase association. Validating the complete POT is not required.

Documentation:

  • Improve phase association description. Use formal notation.

Daniel Armbruster added 7 commits June 21, 2022 15:48
Allow up to 4 samples offset. This should be enough if a detector covers
a single sensor location.
Implement the arrival offset validation according to the documentation,
i.e. the pick associated with the template waveform processor T_k is
associated if either the POT's row or the POT's column w.r.t. T_k is
valid. This reduces the time complexity to O(n).
There is no need to enable the POT entries over and over again, since

- `createPOT()` creates a POT where all entries are enabled
- the linker does not disable any entries.
@damb damb added enhancement New feature or request detection Refers to detection related topics labels Jun 24, 2022
@damb damb self-assigned this Jun 24, 2022
@damb damb merged commit 3520c7e into master Jun 24, 2022
@damb damb deleted the feature/speedup-linking branch June 24, 2022 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
detection Refers to detection related topics enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant