tests for sketch_symmetric

BallisticLA · Jun 27, 2024 · 7320f0a · 7320f0a
1 parent 5fe8f87
commit 7320f0a
Show file tree

Hide file tree

Showing 3 changed files with 296 additions and 35 deletions.
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -17,6 +17,7 @@ if (GTest_FOUND)
         test_matmul_cores/test_rskges.cc
 
         test_matmul_wrappers/test_sketch_vector.cc
+        test_matmul_wrappers/test_sketch_symmetric.cc
     )
     target_link_libraries(RandBLAS_tests
         RandBLAS

diff --git a/test/test_matmul_wrappers/DevNotes.md b/test/test_matmul_wrappers/DevNotes.md
@@ -1,13 +1,16 @@
 # Notes on functionality tested in test_matmul_wrappers
 
+
 ## Tests for sketch_sparse
 
 
 ## Tests for sketch_symmetric
-
+Right now this falls back on sketch_general, which means it suffices to 
+test with only DenseSkOp.
 
 ## Tests for sketch_vector
-Right now sketch_vector falls back on sketch_general.
+Right now sketch_vector falls back on sketch_general, which means it suffices
+to test with only DenseSkOp.
 There's an argument to be made for it to directly handle
 dispatching of GEMV (for DenseSkOp) and SPMV (for SparseSkOp).
 Additional tests would be warranted if we made that change.