Skip to content

fixing Java wrapper for mixChannels#958

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
apavlenko:fix_java_mixchannels
Jun 10, 2013
Merged

fixing Java wrapper for mixChannels#958
opencv-pushbot merged 1 commit intoopencv:masterfrom
apavlenko:fix_java_mixchannels

Conversation

@apavlenko
Copy link
Copy Markdown
Contributor

'dst' should be treated as an input arg (epilog not needed)

@ghost ghost assigned SpecLad Jun 6, 2013
@apavlenko
Copy link
Copy Markdown
Contributor Author

Roman, could you review?

@SpecLad
Copy link
Copy Markdown

SpecLad commented Jun 6, 2013

This feels too hacky. It should either be a separate table { class : { func : {arg_name : [attrib]} } } or the current one should be changed to something like { class : { func : {arg_name : {"ctype": ctype, "attrib": attrib}} } }.

But actually, the root cause of this bug is a deficiency in the argument type. This argument is not really an InputOutputArrayOfArrays, but rather something like InputArrayOfOutputArrays (which we don't have). Maybe we should just introduce this new type and not special-case this particular argument?

'dst' should be treated as an input arg (epilog not needed)
@SpecLad
Copy link
Copy Markdown

SpecLad commented Jun 10, 2013

👍

@opencv-pushbot opencv-pushbot merged commit 438649f into opencv:master Jun 10, 2013
savuor pushed a commit to nickyu-zhu/opencv that referenced this pull request Oct 27, 2023
…amic_shape

Use YuNet of fixed input shape to fix not-supported-dynamic-zero-shape for FaceDetectorYN
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