Skip to content

Add note about compute sanitizer for ccx.Device() #733

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
Closed
ZzEeKkAa wants to merge 2 commits into from
Closed

Add note about compute sanitizer for ccx.Device() #733

ZzEeKkAa wants to merge 2 commits into from

Conversation

@ZzEeKkAa
Copy link
Contributor

@ZzEeKkAa ZzEeKkAa commented Jun 27, 2025 •
edited
Loading

Description

Add note on using ccx.Device() with compute-sanitizer.

Related issue: #562

Checklist

  • I am familiar with the Contributing Guidelines.
  • n/a New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@github-project-automation github-project-automation bot moved this to Todo in CCCL Jun 27, 2025
@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jun 27, 2025

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@leofang
Copy link
Member

leofang commented Jun 27, 2025

Thanks, Yevhenii! I think this impacts to all cuda.core objects, not just Device. I am thinking we should create a new "Tips & Tricks" page like what we did for cuda.bindings, and move this note there. WDYT?

@leofang leofang self-requested a review June 27, 2025 14:57
@leofang leofang added this to the cuda.core beta 5 milestone Jun 27, 2025
@leofang leofang added documentation Improvements or additions to documentation P1 Medium priority - Should do cuda.core Everything related to the cuda.core module labels Jun 27, 2025
@ZzEeKkAa
Copy link
Contributor Author

ZzEeKkAa commented Jun 27, 2025

Yeah, we can do that. I'm wondering how to do it the way, so if user face this problem, they will navigate to the correct place...

@leofang
Copy link
Member

leofang commented Jun 28, 2025

What do you mean?

cpcloud
cpcloud approved these changes Oct 30, 2025
View reviewed changes
Copy link
Contributor

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

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

This seems fine to merge after fixing up conflicts.

Extending this to other objects/writing up tips and tricks can be done in a follow-up.

@github-project-automation github-project-automation bot moved this from Todo to In Review in CCCL Oct 30, 2025
@cpcloud
Copy link
Contributor

cpcloud commented Nov 10, 2025

@ZzEeKkAa Can you fix conflicts so we can merge?

@leofang
Copy link
Member

leofang commented Nov 10, 2025

I suggest we close this and work on #1223. It does not make sense to add such a note to the device class.

@leofang leofang closed this Nov 13, 2025
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cpcloud cpcloud cpcloud approved these changes

@leofang leofang Awaiting requested review from leofang

Assignees

@ZzEeKkAa ZzEeKkAa

Labels

cuda.core Everything related to the cuda.core module documentation Improvements or additions to documentation P1 Medium priority - Should do

Projects

CCCL
Status: Done

Milestone

cuda.core backlog

Development

Successfully merging this pull request may close these issues.

3 participants

@ZzEeKkAa @leofang @cpcloud