Skip to content

[opencv4] Fix cuDNN feature for CUDA 12.2#33898

Merged
vicroms merged 2 commits intomicrosoft:masterfrom
pskopnik:fix-33874-opencv4-cuda12.2
Sep 26, 2023
Merged

[opencv4] Fix cuDNN feature for CUDA 12.2#33898
vicroms merged 2 commits intomicrosoft:masterfrom
pskopnik:fix-33874-opencv4-cuda12.2

Conversation

@pskopnik
Copy link
Copy Markdown
Contributor

@pskopnik pskopnik commented Sep 20, 2023

Fixes #33874.

Adds fix from opencv/opencv#24104 as a patch to the opencv4 v4.8.0 port, to allow installing with dnn-cuda feature and CUDA 12.2.

  • Changes comply with the maintainer guide
  • SHA512s are updated for each updated download
  • The "supports" clause reflects platforms that may be fixed by this new version
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@pskopnik pskopnik changed the title [opencv4] Fix 33874 opencv4 cuda12.2 [opencv4] Fix cuDNN feature for CUDA 12.2 Sep 20, 2023
@pskopnik
Copy link
Copy Markdown
Contributor Author

pskopnik commented Sep 20, 2023

@microsoft-github-policy-service agree

@jimwang118 jimwang118 added the category:port-bug The issue is with a library, which is something the port should already support label Sep 20, 2023
@pskopnik pskopnik marked this pull request as ready for review September 20, 2023 14:07
@pskopnik
Copy link
Copy Markdown
Contributor Author

pskopnik commented Sep 20, 2023

Built successfully locally

@jimwang118
Copy link
Copy Markdown
Contributor

jimwang118 commented Sep 21, 2023

If there is a code conflict, please fix it.

@pskopnik
Copy link
Copy Markdown
Contributor Author

pskopnik commented Sep 21, 2023

Changes rebased on top of 4.8.0#6 added to master yesterday

@jimwang118
Copy link
Copy Markdown
Contributor

jimwang118 commented Sep 22, 2023

Usage test pass with following triplets:

x86-windows
x64-windows

@jimwang118 jimwang118 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Sep 22, 2023
@vicroms vicroms merged commit 70fd7df into microsoft:master Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jimwang118 jimwang118

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[opencv4] Build error with CUDA 12.2

3 participants

@pskopnik @jimwang118 @vicroms