-
Notifications
You must be signed in to change notification settings - Fork 54
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.
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
Sparse matrix (DCSR_matrix) multiplication #1251
base: main
Are you sure you want to change the base?
Conversation
* add pr workflow * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update README.md * Update .gitlab-ci.yml --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ytics/heat into create-pull-request/patch
for more information, see https://pre-commit.ci
…to sparse-matmul-impl
…lytics/heat into sparse-matmul-impl
Thank you for the PR! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1251 +/- ##
==========================================
- Coverage 91.75% 91.70% -0.05%
==========================================
Files 77 79 +2
Lines 11080 11094 +14
==========================================
+ Hits 10166 10174 +8
- Misses 914 920 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This pull request is stale because it has been open for 60 days with no activity. |
Due Diligence
main
for new features, latest release branch (e.g.release/1.3.x
) for bug fixesDescription
A very basic implementation of sparse matrix multiplication using torch's matmul operations.
Not efficient because the tensors are collected before the operation and then redistributed.
Still exploring more efficient parallel solutions.
Type of change
New feature
Does this change modify the behaviour of other functions? If so, which?
no