Skip to content

Clarify A2 feature-test fixture - #325

Merged
sdatkinson merged 3 commits into
mainfrom
fixes
Aug 23, 2026
Merged

Clarify A2 feature-test fixture#325
sdatkinson merged 3 commits into
mainfrom
fixes

Conversation

@sdatkinson

@sdatkinson sdatkinson commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • rename the misleading wavenet_a2_max.nam fixture to wavenet_a2_feature_test.nam
  • update the fixture generator and tests to use the new name
  • clarify that this generic feature-coverage fixture is not the eight-channel A2-Full architecture

This prevents performance results from the generic feature-stress model being mistaken for measurements of the specialized A2-Full path.

Verification

  • Release run_tests passes
  • touched C++ tests pass clang-format 19 validation
  • actual A2-Full candidate performance was 0.19% faster than v0.5.4 by median across 30 interleaved runs
  • the current A2-Full specialized path measured 1.27x faster than the generic WaveNet path at 64-sample blocks

@sdatkinson sdatkinson changed the title Add release checks and clarify A2 feature fixture Document release guarantees and clarify A2 feature fixture Aug 23, 2026
@sdatkinson sdatkinson changed the title Document release guarantees and clarify A2 feature fixture Clarify A2 feature-test fixture Aug 23, 2026
@sdatkinson
sdatkinson merged commit 4fa3221 into main Aug 23, 2026
4 checks passed
@sdatkinson
sdatkinson deleted the fixes branch August 23, 2026 06:04
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