Skip to content

test: expand broadcast operation coverage for issue #301#582

Open
thani-ath-nain wants to merge 1 commit intogorgonia:masterfrom
thani-ath-nain:issue-301-broadcast-tests
Open

test: expand broadcast operation coverage for issue #301#582
thani-ath-nain wants to merge 1 commit intogorgonia:masterfrom
thani-ath-nain:issue-301-broadcast-tests

Conversation

@thani-ath-nain
Copy link
Copy Markdown

@thani-ath-nain thani-ath-nain commented Apr 13, 2026

Summary

  • fill missing broadcast test cases in operations_broadcast_test.go (including 3D cases)
  • add tests for other Broadcast* wrappers (Sub, HadamardDiv, Pow, Lt/Gt/Lte/Gte/Eq/Ne)
  • add extra Broadcast edge-path tests and Lift2Broadcast tests

Validation

  • gofmt on changed files
  • ASSUME_NO_MOVING_GC_UNSAFE_RISK_IT_WITH=go1.26 go test .

Note

A few formatting changes have been introduced as well, like using different notation for octal numbers. Happy to make effort to take them out if needed.

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