Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable GraphBLAS binding to write arrays directly #36

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Enable GraphBLAS binding to write arrays directly #36

merged 1 commit into from
Jul 10, 2023

Commits on Jul 10, 2023

  1. Enable GraphBLAS binding to write arrays directly 

    Previous implementation would always write a coordinate file due to GraphBLAS iterators sometimes throwing GrB_NOT_IMPLEMENTED. Turns out that only throws when the matrix is GxB_BY_ROW, as Matrix Market requires column-major order and GraphBLAS appears to only support iterating on the native direction.

Note that this change is only efficient if the matrix is GxB_BY_COL as that is the only case where an iterator can be used. If the matrix is GxB_BY_ROW, then a full copy and sort is necessary, a very inefficient operation, but still better than writing a coordinate version of that matrix.
    @alugowski
    alugowski committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    346be14 View commit details
    Browse the repository at this point in the history