8000 Use matrixmultiply cgemm and improve tests by bluss · Pull Request #1118 · rust-ndarray/ndarray · GitHub
[go: up one dir, main page]

Skip to content

Use matrixmultiply cgemm and improve tests#1118

Merged
bluss merged 2 commits intomasterfrom
matmul-complex
Nov 21, 2021
Merged

Use matrixmultiply cgemm and improve tests#1118
bluss merged 2 commits intomasterfrom
matmul-complex

Conversation

@bluss
Copy link
Member
@bluss bluss commented Nov 21, 2021

Refactor tests to use generics in a bit smarter way.

We test both .dot() and general_mat_mul. Tests now use relative accuracy
more explicitly (this works better with generics, instead of using
approx).

This follows up previous change #1106 by improving the default/
fallback for Complex<f32> and Complex<f64> by using
matrixmultiply.

This performance improvement is already good (2-3x on small problems),
but enabling BLAS should be much better.

@bluss bluss force-pushed the matmul-complex branch 2 times, most recently from 02f511c to 37eae14 Compare November 21, 2021 12:06
Refactor tests to use generics in a bit smarter way.

We test both .dot() and general_mat_mul. Tests now use relative accuracy
more explicitly (this works better with generics, instead of using
approx).
@bluss
Copy link
Member Author
bluss commented Nov 21, 2021

Here I wonder if complex support in general should be optional and opt-in, might be a less used feature.

@bluss bluss merged commit 5e6ebf5 into master Nov 21, 2021
@bluss bluss deleted the matmul-complex branch November 21, 2021 16:21
@bluss bluss added this to the 0.15.4 milestone Nov 22, 2021
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

0