Skip to content

Use new faster auto lowpp class implementation #1244

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
Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Use new faster auto lowpp class implementation #1244

wants to merge 7 commits into from
+1,266 −788

Conversation

@mdboom
Copy link
Contributor

@mdboom mdboom commented Nov 17, 2025

This is still somewhat a WIP, but I would like to get more CI.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Nov 17, 2025

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

Contributors can view more details about this message here.

@mdboom
Copy link
Contributor Author

mdboom commented Nov 17, 2025

/ok to test

@github-actions
Copy link

github-actions bot commented Nov 17, 2025

Doc Preview CI

🚀 View preview at
https://nvidia.github.io/cuda-python/pr-preview/pr-1244/

https://nvidia.github.io/cuda-python/pr-preview/pr-1244/cuda-core/

https://nvidia.github.io/cuda-python/pr-preview/pr-1244/cuda-bindings/

https://nvidia.github.io/cuda-python/pr-preview/pr-1244/cuda-pathfinder/


Preview will be ready when the GitHub Pages deployment is complete.

@mdboom mdboom force-pushed the fast-auto-lowpp-class2 branch from 468f640 to 06930f5 Compare November 17, 2025 16:55
@mdboom mdboom force-pushed the fast-auto-lowpp-class2 branch from 06930f5 to f01c309 Compare November 17, 2025 19:14
@mdboom
Copy link
Contributor Author

mdboom commented Nov 17, 2025

/ok to test

leofang
leofang reviewed Nov 17, 2025
View reviewed changes
leofang
leofang reviewed Nov 17, 2025
View reviewed changes
leofang
leofang reviewed Nov 17, 2025
View reviewed changes
leofang
leofang reviewed Nov 17, 2025
View reviewed changes
kkraus14
kkraus14 reviewed Nov 17, 2025
View reviewed changes
@leofang leofang added enhancement Any code-related improvements P0 High priority - Must do! cuda.bindings Everything related to the cuda.bindings module labels Nov 18, 2025
@leofang leofang added this to the cuda-python 13-next, 12-next milestone Nov 18, 2025
@mdboom
Copy link
Contributor Author

mdboom commented Nov 21, 2025

/ok to test

@mdboom
Copy link
Contributor Author

mdboom commented Nov 24, 2025

/ok to test

@mdboom
Copy link
Contributor Author

mdboom commented Nov 24, 2025

/ok to test

leofang
leofang reviewed Nov 24, 2025
View reviewed changes
Copy link
Member

@leofang leofang left a comment

Choose a reason for hiding this comment

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

Left a few quick questions, but otherwise LGTM. I think this PR can be un-drafted now? @mdboom do you plan to add more changes? Maybe you want to piggyback the dtype fix here?

@leofang leofang mentioned this pull request Nov 27, 2025
Closed
@mdboom
Copy link
Contributor Author

mdboom commented Dec 1, 2025

/ok to test

@mdboom
Copy link
Contributor Author

mdboom commented Dec 1, 2025

/ok to test

@mdboom
Copy link
Contributor Author

mdboom commented Dec 1, 2025

/ok to test

@leofang leofang mentioned this pull request Dec 1, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kkraus14 kkraus14 kkraus14 left review comments

@leofang leofang leofang approved these changes

Assignees

@mdboom mdboom

Labels

cuda.bindings Everything related to the cuda.bindings module enhancement Any code-related improvements P0 High priority - Must do!

Projects

None yet

Milestone

cuda-python 13-next, 12-next

Development

Successfully merging this pull request may close these issues.

3 participants

@mdboom @kkraus14 @leofang