Skip to content

Conversation

@mdboom
Copy link
Contributor

@mdboom mdboom commented Jan 8, 2026

Now that the skeleton of cuda.core.system is merged, I'm starting the process of methodically adding APIs. This adds the one remaining "system-level" API, nvmlSystemGetDriverBranch.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 8, 2026

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.

@mdboom mdboom requested review from Copilot and cpcloud January 8, 2026 15:11
@mdboom
Copy link
Contributor Author

mdboom commented Jan 8, 2026

/ok to test

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new system-level API get_driver_branch() that retrieves the driver branch of the NVIDIA driver installed on the system by wrapping the NVML function nvmlSystemGetDriverBranch.

  • Adds get_driver_branch() function to _system.pyx
  • Adds test coverage for the new function
  • Updates __all__ exports to include the new function

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
cuda_core/cuda/core/system/_system.pyx Implements get_driver_branch() function and adds it to __all__ exports
cuda_core/tests/system/test_system_system.py Adds test case for get_driver_branch() function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

mdboom and others added 2 commits January 8, 2026 10:17
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mdboom
Copy link
Contributor Author

mdboom commented Jan 8, 2026

/ok to test

@mdboom mdboom changed the title Add system-level APIs cuda.core.system: Add system-level APIs Jan 8, 2026
@mdboom mdboom self-assigned this Jan 8, 2026
@mdboom mdboom added enhancement Any code-related improvements feature New feature or request cuda.core Everything related to the cuda.core module labels Jan 8, 2026
@github-actions

This comment has been minimized.

@kkraus14 kkraus14 merged commit f8dddb3 into NVIDIA:main Jan 9, 2026
143 of 144 checks passed
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

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

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 enhancement Any code-related improvements feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants