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

nv-filters: Updates for new SDK #11814

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pkviet
Copy link
Member

@pkviet pkviet commented Feb 1, 2025

Description

This PR does two things:

  • updates the checks against the new redistributables of NVIDIA AUDIO & VIDEO effects to:

    • 7.6.0 for Video Effects;
    • 1.6.1.2 for Audio Effects.
  • cleans up the code related with cuda RT functions, which is not used anymore.

Motivation and Context

Allow nv-filters to work with newer SDK.
This also fixes bug: #11813

How Has This Been Tested?

The video & audio effects load fine on windows 11 PRO 24H2.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Code cleanup (non-breaking change which makes code smaller or more readable)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@pkviet pkviet added Seeking Testers Build artifacts on CI Bug Fix Non-breaking change which fixes an issue labels Feb 1, 2025
@pkviet pkviet added this to the OBS Studio 31.1 milestone Feb 1, 2025
This updates checks against current SDK redist versions:
- 0.7.6 (Video Effects);
- 1.6.1.2 (Audio Effects).
Users will receive a warning if their redist versions is smaller than
these.

Signed-off-by: pkv <[email protected]>
We don't use any more cuda RT functions.
So this commit removes their loading from the cudaRT dll.
This fixes also a crash when loading different versions of cudaRT when
the SDK is updated [1].

[1] obsproject#11813

Signed-off-by: pkv <[email protected]>
@pkviet pkviet added the Code Cleanup Non-breaking change which makes code smaller or more readable label Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix Non-breaking change which fixes an issue Code Cleanup Non-breaking change which makes code smaller or more readable Seeking Testers Build artifacts on CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant