Skip to content

Issues 20260326#170

Merged
simoninns merged 8 commits intomainfrom
issues-20260326
Mar 26, 2026
Merged

Issues 20260326#170
simoninns merged 8 commits intomainfrom
issues-20260326

Conversation

@simoninns
Copy link
Copy Markdown
Owner

Summary

This PR delivers a full vectorscope upgrade across core, presenter/view types, and GUI, and fixes the final integration issue in the branch.

Main changes included in this branch:

Coverage and regression protection added in this branch:

  • New core vectorscope analysis tests.
  • Expanded preview type and preview view registry tests.
  • New GUI vectorscope geometry unit tests.
  • Build/test wiring updates so new test targets are included.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other

Validation

Executed locally on this branch:

  • cmake --build build -j
  • ctest --test-dir build --output-on-failure

Result:

  • 653/653 tests passed
  • 0 failed
  • Total test time: 15.28s

Checklist

  • Scope is focused and minimal
  • Build passes locally
  • Related docs were updated if needed
  • Linked issue (if applicable)

Related issue

Closes #81, #82, #160 and #167

@simoninns simoninns merged commit 6656c34 into main Mar 26, 2026
14 checks passed
@simoninns simoninns deleted the issues-20260326 branch March 26, 2026 15:32
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.

Vectorscope doesn't look correct for NTSC

1 participant