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

5.x merge 4.x #3819

Merged
merged 4 commits into from
Nov 5, 2024
Merged

5.x merge 4.x #3819

merged 4 commits into from
Nov 5, 2024

Conversation

asmorkalov
Copy link
Contributor

Main OpenCV: opencv/opencv#26404

#3800 from vrabaud:cuda
#3815 from cudawarped:cuda_fix_python_binaryop_with_scalar

Previous "Merge 4.x": #3793

cudawarped and others added 4 commits October 29, 2024 15:42
Get CUDA code to compile with clang CUDA and without CUDA opencv#3800

Changelist:
- there are some syntactic changes: `<< <` -> `<<<`. For some reason, I do not need to change all those in the code.
-  `::min` -> `std::min` in `__host__` code
- `modules/cudaimgproc/src/moments.cpp` needs to have the CUDA code in the `#ifdef`
- The signature of `cv::cuda::swapChannels` is not exactly the same as the C++ one in `modules/cudaimgproc/src/color.cpp` 
- `cv::cuda::FarnebackOpticalFlow::create` needs to be explicit about which FarnebackOpticalFlow it returns

### Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
…op_with_scalar

`cudaarithm`: fix python bindings for binary ops involving scalars
@asmorkalov asmorkalov merged commit bd3b734 into opencv:5.x Nov 5, 2024
2 of 9 checks passed
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 this pull request may close these issues.

3 participants