Naming conventions for basic sketching #47
Replies: 6 comments 6 replies
-
Option 1. This isn't worth considering. I'm just keeping this up to document the discussion process.
The first character ( The type of the sketching operator could be incorporated into the function name with a single signifying character, such as |
Beta Was this translation helpful? Give feedback.
-
Option 2.
|
Beta Was this translation helpful? Give feedback.
-
Option 3.
The A function like |
Beta Was this translation helpful? Give feedback.
-
This comment is a short reference. Refer to later comments for details and thought processes. Notes that apply to all options
Summary of optionsOption 1 isn't worth considering. Option 2
Option 2A
Option 3
Option 3A
There's also Option 3B and Option 3C. I don't think those are worth considering. |
Beta Was this translation helpful? Give feedback.
-
Prompt
Options
FeedbackI am in favor of option 3. |
Beta Was this translation helpful? Give feedback.
-
The table of naming conventions for the C++ linear algebra standard is here: https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/p1673r11.html#overview-of-contents (at the end of section 4). |
Beta Was this translation helpful? Give feedback.
-
RandBLAS's support for basic sketching is taking shape. We have code and tests for any combination of sketching from the left or right and using a dense or sparse sketching operator. That is, we have code to compute
where$S$ is dense or sparse and $A$ is a general dense matrix.
This "GitHub Discussions" thread documents proposals for the naming of these functions (and similar functions) in RandBLAS.
Requirements
Where the discussion stands
The latest update to RandBLAS/main uses Option 2 below. RandBLAS/main also has older code that follows Option 1. We'll make these things consistent once we come to a consensus here.
I'm currently leaning towards Option 3 (not 3A). There's less ambiguity in how one might pronounce the constituent abbreviations compared to Option 3A. It has the
ramm
prefix to mimicgemm
. The natural suffixesgefr
,syfr
,hefl
, etc.. are readable insofar as the letter "f" signifies "from". The fully qualified names that includesparse
orlevel3
should see limited use. As a result, we wouldn't want to abbreviate the sketching operator type in the manner suggested by Option 3B.Things to keep in mind
_Generic
.We might converge on a naming convention for basic sketching, and yet still need to revisit this when we get to fancier things such as multi-sketching.
Beta Was this translation helpful? Give feedback.
All reactions