You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Error in the documentation of the function "any" #147
I just spent quite some time figuring out why my code did not produce the expected results, and just found out the reason. In this package's documentation, I found next to the any operator:
any(x, y) = 1 if x or y are stored values
which is the usual behavior of the operation any on an iterable collection in Julia.
However, in the SuiteSparseGraphBLAS user guide (which I found there) , we are told page 65:
The ANY operator is very unusual, but very powerful. It is the function
fany(x,y) = x, or y, where GraphBLAS has to freedom to select either x,
or y, at its own discretion. Do not confuse the ANY operator with the any
function in MATLAB/Octave, which computes a reduction using the logical
OR operator.
From what I've seen, the behavior we are getting is the one from SuitedSparseGraphBLAS, so it could be nice updating the docs to avoid this confusion.
@ViralBShah I don't know if I'll have other additions like this in the future, I'm mostly working on other implementations of GraphBLAS and using this package to compare. But I'll keep submitting fixes if I find small issues!
Thank you for the merge, I'll mark this as closed :))
Activity
AntoineBut commentedon May 12, 2025
Additional information : the note next to the
pairoperator had an error too, which is also fixed in this quick PR :))ViralBShah commentedon Jul 17, 2025
@AntoineBut I'm not sure of your interest level, but if you would like to maintain and update this package, I'm happy to give you commit access.
AntoineBut commentedon Oct 4, 2025
@ViralBShah I don't know if I'll have other additions like this in the future, I'm mostly working on other implementations of GraphBLAS and using this package to compare. But I'll keep submitting fixes if I find small issues!
Thank you for the merge, I'll mark this as closed :))