Skip to content

Pair imported wave paths with archived MATLAB fixtures - #327

Merged
cmccomb merged 1 commit into
masterfrom
test/imported-wave-matlab-fixtures
Oct 14, 2026
Merged

cmccomb merged 1 commit into
masterfrom
test/imported-wave-matlab-fixtures

Conversation

@cmccomb

@cmccomb cmccomb commented Oct 14, 2026

Copy link
Copy Markdown
Member

Summary

Run the production WaveClass against the repository's archived MATLAB-generated imported-wave fixtures. Both .txt and .mat paths now gate imported elevation interpolation and startup ramping over 400 s, plus imported spectral bins, phases, and synthesized elevation over 200 s.

These historical fixtures add a numerical regression check for the import paths. They are not a new comparison to the currently pinned WEC-Sim revision, so published-case badges do not change.

Verification

  • Focused tests: 4 passed.
  • Full local Python suite: 267 passed, 333 skipped; combined line-and-branch coverage rose from 78.25% to 78.29%, still displayed as 78%.
  • Maximum imported-elevation history difference: 1.55e-13 m; maximum imported-spectrum elevation difference: 2.85e-14 m, below 2e-12 m and 1e-12 m gates.

@cmccomb
cmccomb merged commit 92537c9 into master Oct 14, 2026
1 check passed
@cmccomb
cmccomb deleted the test/imported-wave-matlab-fixtures branch October 14, 2026 00:41
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.

1 participant