Skip to content

[doc-only] Fix memcpy entry in driver_v2 API catalog - #3385

Open
mdboom wants to merge 1 commit into
NVIDIA:mainfrom
mdboom:issue3381
Open

mdboom wants to merge 1 commit into
NVIDIA:mainfrom
mdboom:issue3381

Conversation

@mdboom

@mdboom mdboom commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

The autosummary list in driver_v2.rst named cu_memcpy, but the actual export is cuda.bindings._v2.driver.memcpy. Sphinx failed the import and silently dropped the entry and its detail page. This renames the entry to memcpy.

I checked the other names in the file's autosummary lists against cuda/bindings/_v2/driver.pyx. They all resolve; a few are aliases.

I haven't done a local docs build, so the memcpy row and detail page appearing and the import warning going away are unverified. The docs job in CI should confirm it.

Fixes #3381

Shipping the correction for the already-tagged 13.5.0 docs through the docs-fix process is not covered here.

🤖 Generated with Claude Code

The autosummary list in driver_v2.rst named `cu_memcpy`, but the actual
export is `cuda.bindings._v2.driver.memcpy`. Sphinx failed the import
and silently dropped the entry and its detail page.

Fixes NVIDIA#3381

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the cuda.bindings Everything related to the cuda.bindings module label Nov 5, 2026
@mdboom mdboom self-assigned this Nov 5, 2026
@mdboom mdboom added this to the cuda.bindings next milestone Nov 5, 2026
@coderabbitai

coderabbitai Bot commented Nov 5, 2026

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

  • Documentation
    • Updated the Memory Management reference to list memcpy.

The driver v2 autosummary entry changes from cu_memcpy to memcpy.

Changes: Driver v2 API catalog

Layer / File(s) Summary
Rename memcpy autosummary entry
cuda_bindings_13/docs/source/module/driver_v2.rst
The autosummary entry changes from cu_memcpy to memcpy.

Assessment against linked issues

Objective Addressed Explanation
Correct the v2 driver catalog name for the memcpy export [#3381] ✅

Possibly related issues

Possibly related PRs

Suggested labels: documentation, cuda.bindings, bug

Suggested reviewers: juenglin

Change: Bug fix · Severity of issue fixed: Low

Merge Risk: Minimal · up to 55d50

This documentation-only change has no identified current-head risk. Docs rendering remains for CI confirmation.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 3429cba and 55d50cc.

Files selected for processing: 1

  • cuda_bindings_13/docs/source/module/driver_v2.rst

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review.

@mdboom mdboom added the documentation Improvements or additions to documentation label Nov 5, 2026
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindings Everything related to the cuda.bindings module documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC]: cuda.bindings._v2.driver.memcpy missing from driver_v2 API catalog

1 participant