Skip to content

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Dec 11, 2025

Add a size property to StridedMemoryView to avoid having dig into layout to get this information.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Dec 11, 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.

@cpcloud
Copy link
Contributor Author

cpcloud commented Dec 11, 2025

/ok to test

@cpcloud cpcloud requested a review from mdboom December 11, 2025 20:29
@github-actions

This comment has been minimized.

@cpcloud cpcloud enabled auto-merge (squash) December 12, 2025 12:18
@cpcloud cpcloud merged commit 4f910cb into NVIDIA:main Dec 12, 2025
138 of 140 checks passed
@cpcloud cpcloud deleted the size-property branch December 12, 2025 21:00
@github-actions
Copy link

Doc Preview CI
Preview removed because the pull request was closed or merged.

@leofang leofang added this to the cuda.core beta 10 milestone Dec 14, 2025
@leofang leofang added P0 High priority - Must do! feature New feature or request cuda.core Everything related to the cuda.core module labels Dec 14, 2025
rwgk added a commit to rwgk/cuda-python that referenced this pull request Dec 15, 2025
Merged latest changes from main including:
- Added size property to StridedMemoryView (NVIDIA#1363)
- NVML bindings updates (NVIDIA#1360)
- cuda_pathfinder pixi configuration (NVIDIA#1365, NVIDIA#1361)

All changes merged cleanly with no conflicts.
@leofang leofang mentioned this pull request Dec 15, 2025
2 tasks
copy-pr-bot bot pushed a commit that referenced this pull request Dec 15, 2025
cpcloud pushed a commit that referenced this pull request Dec 15, 2025
* Revert "feat: add `size` property to `StridedMemoryView` (#1363)"

This reverts commit 4f910cb.

* Revert "feat: Introduce StridedLayout, support wrapping external allocations in Buffer, add StridedMemoryView.from_buffer (#1283)"

This reverts commit 0eeabc1.

* re-implement SMV.size without using SL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.core Everything related to the cuda.core module feature New feature or request P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants