Closed jim22k closed 2 years ago
Totals | |
---|---|
Change from base Build 1387293998: | -2.9% |
Covered Lines: | 3219 |
Relevant Lines: | 3923 |
This looks like a fine approach for now, but I think we may want to make graphblas-exec
an actual Python executable that we can run outside of llvm-lit at the command line. That will make it easier to debug test issues in the long run.
This only converts 2 lowering tests to functional tests, but shows the pattern.
Before converting more, we should upgrade
graphblas.print
to display a sparse tensor in a consistent format. Or possibly add a new operationgraphblas.print_tensor
with options to control how much is displayed: