Skip to content
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.

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

Add legacy-functions feature to wrapper and bindings #39

Merged
merged 2 commits into from
Jan 20, 2023

Conversation

Cldfire
Copy link
Collaborator

@Cldfire Cldfire commented Nov 26, 2022

This makes it possible to enable a feature and then have access to older versions of functions in both the wrapper and the bindings.

Initially adding wrappers for the v2 versions of running compute / graphics processes.

Closes #33.

@Cldfire
Copy link
Collaborator Author

Cldfire commented Jan 5, 2023

@KisaragiEffective bump, no worries if you're busy but would love to get your thoughts before merging this :)

Copy link
Contributor

@KisaragiEffective KisaragiEffective left a comment

Choose a reason for hiding this comment

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

Hi, it's been a while (I've been on the edge in last month). This change looks good for me :) (compiles on both local and my CI -- KisaragiEffective/nikka-leaps#20)

Thank you for your efforts and applying my (initial) suggestion.

@Cldfire
Copy link
Collaborator Author

Cldfire commented Jan 6, 2023

Nice, thanks for giving it a go! I'll get this out in a release when I have the opportunity.

Unfortunately there is no way to compile but not run doctests in CI only
@Cldfire Cldfire merged commit e12f4bd into master Jan 20, 2023
@Cldfire Cldfire deleted the legacy-functions branch January 20, 2023 22:59
@Cldfire
Copy link
Collaborator Author

Cldfire commented Jan 21, 2023

This has been released :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-compatibility Related to compatibility with older versions of NVML enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature request: add support for nvmlDeviceGetGraphicsRunningProcesses_v2
2 participants