Skip to content

test: cover ts_cov window semantics - #70

Merged
initial-d merged 1 commit into
initial-d:mainfrom
Angelo9up:test-ts-cov
Sep 17, 2026
Merged

initial-d merged 1 commit into
initial-d:mainfrom
Angelo9up:test-ts-cov

Conversation

@Angelo9up

Copy link
Copy Markdown
Contributor

Summary

  • add direct coverage for ts_cov trailing-window semantics
  • verify sample covariance values and front-padding output masks

Validation

  • pytest tests\test_tensor_factors.py -q — 10 passed
  • pytest -q — all tests passed
  • ruff check tests\test_tensor_factors.py — passed
  • git diff --check — passed (Git only reported the Windows LF/CRLF conversion warning)

This is a focused test-only change; no production implementation is modified.

@initial-d initial-d left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Angelo9up. This is another focused and useful primitive-semantics test: it verifies s_cov uses trailing sample covariance and preserves the expected front-padding mask behavior. I checked the math against the implementation and verified targeted tests, full local pytest, ruff, diff check, and the GitHub CI matrix.

@initial-d
initial-d merged commit 0f0c8bf into initial-d:main Sep 17, 2026
3 checks passed
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.

2 participants