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

cudaimgproc fails to build with cuda 12.4 #3741

Closed
4 tasks done
IllogicalMoose opened this issue May 23, 2024 · 0 comments · Fixed by #3742
Closed
4 tasks done

cudaimgproc fails to build with cuda 12.4 #3741

IllogicalMoose opened this issue May 23, 2024 · 0 comments · Fixed by #3742

Comments

@IllogicalMoose
Copy link

IllogicalMoose commented May 23, 2024

System information (version)

OpenCV => 4.9.0
Operating System / Platform => Linux
Compiler => clang 13

Detailed description

Similar to issue #3725, in cuda 12.4 for nppi_statistics_functions.h the functions nppiHistogramEvenGetBufferSize_* have changed the hpBufferSize argument from 'int *' to 'size_t *'

Example:
12.3:
NppStatus
nppiHistogramEvenGetBufferSize_8u_C1R(NppiSize oSizeROI, int nLevels ,int * hpBufferSize);

12.4:
NppStatus
nppiHistogramEvenGetBufferSize_8u_C1R(NppiSize oSizeROI, int nLevels ,size_t * hpBufferSize);

This produces the compile error in cudaimgproc:

modules/cudaimgproc/src/histogram.cpp:519: error: value of type 'NppStatus (NppiSize, int, size_t *)' (aka 'NppStatus (NppiSize, int, unsigned long )') is not implicitly convertible to 'get_buf_size_c1_t' (aka 'NppStatus ()(NppiSize, int, int *)')
519 | NppHistogramEvenC1<CV_8U , nppiHistogramEven_8u_C1R , nppiHistogramEvenGetBufferSize_8u_C1R >::hist,

Steps to reproduce

Build cudaimgproc with cuda 12.4

Issue submission checklist
  • I report the issue, it's not a question
  • I checked the problem with documentation, FAQ, open issues,
    forum.opencv.org, Stack Overflow, etc and have not found any solution
  • I updated to the latest OpenCV version and the issue is still there
  • There is reproducer code and related data files: videos, images, onnx, etc
vrabaud added a commit to vrabaud/opencv_contrib that referenced this issue May 23, 2024
asmorkalov pushed a commit to asmorkalov/opencv_contrib that referenced this issue May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant