Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Math.MatMul: should only allow count relationship between columns of first matrix, and lines of second matrix #832

Open
ccfontes opened this issue Oct 24, 2023 · 0 comments

Comments

@ccfontes
Copy link
Contributor

Describe the bug
Matrix product in Math.MatMul allows incompatible count relationship between columns of first matrix, and lines of second matrix.

To Reproduce

[@f4(1 2 3 4) @f4(5 6 7 8)] | Math.MatMul([@f4(9 10 11 12) @f4(13 14 15 16)])
;=> [(59 78 97 116) (83 110 137 164) (0 0 0 0) (0 0 0 0)]

first matrix: columns=4
second matrix: lines=2

Expected behavior
The number of columns in the first matrix should be equal to the number of lines in the second matrix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant