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.

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

[ROCm] conditionally enable hipsparse const descriptors for version >= 2.4.0 #110317

Closed
wants to merge 2 commits into from

Conversation

jeffdaily
Copy link
Collaborator

@jeffdaily jeffdaily commented Sep 29, 2023

This is in preparation for upcoming backwards-incompatible hipsparse changes.

cc @sunway513 @jithunnair-amd @pruthvistony @ROCmSupport @dllehr-amd @jataylo @hongxiayang

…#1217)

* conditionally enable hipsparse const descriptors

* update hipsparse const API version condition to 2.4.0
@pytorch-bot pytorch-bot bot added the module: rocm AMD GPU support for Pytorch label Sep 29, 2023
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 29, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/110317

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit f15f5b3 with merge base 359c2a5 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@jithunnair-amd jithunnair-amd added ciflow/trunk Trigger trunk jobs on your pull request release notes: rocm mandatorylabel labels Oct 2, 2023
@jithunnair-amd
Copy link
Collaborator

@jeffdaily @malfet Adding release notes: rocm label because this PR impacts files installed by PyTorch build. Please remove if you think it's not required.

Copy link
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

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

LGTM if CI is green

@jithunnair-amd jithunnair-amd added the rocm This tag is for PRs from ROCm team label Oct 2, 2023
To prevent compilation errors on non-ROCm platforms.
@jeffdaily
Copy link
Collaborator Author

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

lubosz pushed a commit to lubosz/pytorch that referenced this pull request Jan 4, 2024
…= 2.4.0 (pytorch#110317)

This is in preparation for upcoming backwards-incompatible hipsparse changes.

Pull Request resolved: pytorch#110317
Approved by: https://github.com/malfet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request Merged module: rocm AMD GPU support for Pytorch open source release notes: rocm mandatorylabel rocm This tag is for PRs from ROCm team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants