Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

[Bugfix] Dynamic load NVML symbols for better compatibility #234

Merged
merged 4 commits into from
Nov 22, 2024

Conversation

chang-l
Copy link
Contributor

@chang-l chang-l commented Nov 18, 2024

[File PR here for the record]
Dynamically load NVML symbols for querying GPU fabric info to address incompatibility issues with outdated display drivers.

Copy link

copy-pr-bot bot commented Nov 18, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Contributor

@linhu-nv linhu-nv left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

@BradReesWork BradReesWork added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Nov 21, 2024
@BradReesWork
Copy link
Member

/okay to test

@BradReesWork
Copy link
Member

/okay to test

@BradReesWork BradReesWork added bug Something isn't working and removed improvement Improves an existing functionality labels Nov 22, 2024
@BradReesWork
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 924e9f9 into rapidsai:branch-24.12 Nov 22, 2024
49 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants