Conversation
|
@llschloesser #3777 was merged 1 hour after your comment. So I'll just rebase my PR.
We do some image processing steps before the debayering. Their resulting datatype is f32. So it's not directly from the sensor. |
ec24b9c to
ddb96cd
Compare
|
If you follow the template pattern of the earlier PR you'll get 32f to gray and 4 channel, there is no need to restrict to 3 channel. |
ddb96cd to
d498de9
Compare
Good point, I'll rework and test it. |
d498de9 to
5ffd0d6
Compare
|
Reworked the changes. |
|
@apbr Could you add simple test for the new feature? |
@asmorkalov I could give it a try. Should they be in here? |
|
Yes, as shortcut you can use existing demosaic test and reference data for u8, but scale it for larger range. I just want to be sure that the template implementation does not lead to overflows or other arithmetical issues. |
Extend
cudaimgproc::demosaicingto also allow images with f32 values.Pull Request Readiness Checklist
See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request
Patch to opencv_extra has the same branch name.