Skip to content

Honor configured origin-time match keys - #967

Merged
wangyinz merged 2 commits into
masterfrom
codex/issue-826-origin-time-keys
Aug 23, 2026
Merged

wangyinz merged 2 commits into
masterfrom
codex/issue-826-origin-time-keys

Conversation

@wangyinz

Copy link
Copy Markdown
Member

Fixes #826

This PR makes cached and database-backed origin-time matchers consistently use their configured waveform and source time keys, including nearest-match disambiguation.

Validation includes an independent agent review, real MongoDB default/custom-key integration, 40 contract and neighboring normalization regressions, an exact baseline-red suite, Black, Python 3.12/3.13 py_compile, and diff checks.

This PR is ready for human review. It must not be merged automatically.

@codecov

codecov Bot commented Aug 13, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.20%. Comparing base (a0c0ba2) to head (66522de).

Files with missing lines Patch % Lines
python/mspasspy/db/normalize.py 84.61% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #967   +/-   ##
=======================================
  Coverage   72.20%   72.20%           
=======================================
  Files         182      182           
  Lines       36090    36091    +1     
  Branches     2532     2532           
=======================================
+ Hits        26057    26060    +3     
+ Misses       9127     9125    -2     
  Partials      906      906           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wangyinz
wangyinz marked this pull request as draft August 14, 2026 06:03
@wangyinz
wangyinz marked this pull request as ready for review August 14, 2026 15:46
@wangyinz
wangyinz force-pushed the codex/issue-826-origin-time-keys branch from 594a170 to 66522de Compare August 23, 2026 09:00
@wangyinz
wangyinz merged commit bff4dbd into master Aug 23, 2026
18 checks passed
@wangyinz
wangyinz deleted the codex/issue-826-origin-time-keys branch August 23, 2026 09:24
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.

Honor configured source and data time keys in OriginTime matchers

1 participant