Add COOMatrix and other key classes/functions into RandBLAS namespace; write tests - Githubissues

BallisticLA / RandBLAS

A header-only C++ library for sketching in randomized linear algebra

https://randblas.readthedocs.io/en/latest/

Other

64 stars 4 forks source link

Add COOMatrix and other key classes/functions into RandBLAS namespace; write tests #81

Closed rileyjmurray closed 6 months ago

rileyjmurray commented 6 months ago

Namespace changes

Move some sparse matrix datatypes into top-level RandBLAS::.
Remove the RandBLAS_Testing namespace (currently only used in RandBLAS/test_util.hh; move code in that namespace to more appropriate files in test/...

Testing

Write dedicated tests for sparse-times-dense matmul that don't rely on the tests for sparse sketching operators times dense matrices.
Made code more readable by adding using statements (like using blas::Layout and using RandBLAS::SparseSkOp) so namespaces don't need to be specified as much.