this issue is about pressio/expressions and pressio/ops.
The main focus is on making the corresponding tests better and more complete.
why i don't like the current status
if a new rank-1 expression is added, we have to add a whole new set of tests for it
there is not clear uniformity in how the tests are written. it is a bit of a mess
what i like to explore/have
I like to come up with a design/structure such that
if a new expression is added, we can easily add via a macro or something, or a sinlge func call that will run ALL ops tests without missing any combination of them.
it is more "structured" in how it is organized and better explained so that new people working on it can understand
maybe also optimized a bit in terms of # of files/executables, since right now we have many of them for individual tests that might not be neede
right now, this is hard to do and has to be done kind of repeatedly as seen while doing the work on the column of a tpetra block multivector https://github.com/Pressio/pressio/pull/647
this issue is about
pressio/expressions
andpressio/ops
. The main focus is on making the corresponding tests better and more complete.why i don't like the current status
if a new rank-1 expression is added, we have to add a whole new set of tests for it
there is not clear uniformity in how the tests are written. it is a bit of a mess
what i like to explore/have
I like to come up with a design/structure such that
if a new expression is added, we can easily add via a macro or something, or a sinlge func call that will run ALL ops tests without missing any combination of them.
it is more "structured" in how it is organized and better explained so that new people working on it can understand
maybe also optimized a bit in terms of # of files/executables, since right now we have many of them for individual tests that might not be neede
right now, this is hard to do and has to be done kind of repeatedly as seen while doing the work on the column of a tpetra block multivector https://github.com/Pressio/pressio/pull/647