Skip to content

Conversation

@BenBrock
Copy link
Collaborator

@BenBrock BenBrock commented Mar 21, 2025

@yhmtsai This adds cuSPARSE to the CI. Merge into your PR #40 whenever you are ready.

@BenBrock BenBrock marked this pull request as ready for review March 21, 2025 18:00
@BenBrock BenBrock requested a review from yhmtsai March 21, 2025 18:01
Copy link
Contributor

@yhmtsai yhmtsai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding it!

shell: bash -l {0}
run: |
module load cmake
cmake -B build -DENABLE_CUSPARSE=ON -DCMAKE_PREFIX_PATH=/usr/local/cuda/targets/x86_64-linux/lib/cmake
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is any cuda module available on machine?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have to ask Geri—I think there's just one global installation of CUDA; I don't think there are difficult CUDA modules.

Copy link
Contributor

@yhmtsai yhmtsai Mar 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. one version now should also be good enough for the test

@BenBrock BenBrock merged commit 6e5401e into dev/yhmtsai/cusparse_spmv Mar 24, 2025
11 checks passed
@BenBrock BenBrock deleted the dev/brock/add-cusparse-to-ci branch March 24, 2025 17:07
@BenBrock
Copy link
Collaborator Author

Sorry, thought you had merged it yourself last week—merging now.

yhmtsai pushed a commit that referenced this pull request May 7, 2025
yhmtsai added a commit that referenced this pull request May 15, 2025
* add cusparse spmv and corresponding cmake

* setup CMake to use the same test and example with thrust

* add the cusparse example without thrust

* Add cuSPARSE to CI. (#50)

* add the cuSPARSE into README

* accept list into add_device_test

---------

Co-authored-by: Benjamin Brock <brock@cs.berkeley.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants