Sparse sketching: SASO and LASO or MajorAxis::Short and MajorAxis::Long #54
Unanswered
rileyjmurray
asked this question in
Q&A
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We recently introduced the
.major_axis
member to theDenseSkOp
struct definition, which can take values in theMajorAxis
enumeration. The concept of a major axis also arises in sparse sketching operators. We've been using the phrase "short-axis sparse operator" and "long-axis sparse operator" for what we might otherwise call operators with short and long major axes, respectively. I'm wondering if it makes sense to unify the terminology here and replaceSparseSkOp.family
withSparseSkOp.major_axis
, and replace SASO/LASO with MajorAxis::Short and MajorAxis::Long. One benefit of this is that we could replaceSparseSkOp.vec_nnz
withSparseSkOp.major_axis_vec_nnz
, which has more of a "self-documenting" flavor.Beta Was this translation helpful? Give feedback.
All reactions