need collaborative effort [closed]

asked 2015-10-30 12:23:18 -0500

updated 2015-12-14 07:41:07 -0500

i want to see collaborative power of forum users.

i think documentation needs some corrections and make up like cv::merge ( related issue : 5111 )

we can do some corrections collaboratively.

I suggest a working model like the steps below.

  • declaration of the need ( as i did )
  • someone make an initial PR ( i think we can encourage inexperienced users to do initial PR )
  • we suggest additional corrections or ideas by comments on PR
  • at the and we will confirm the PR image description by this way we will make merging job easier to core developer

i hope i will get response and we make contributions together. also waiting your opinions.

EDIT : I think my question needs a small explanation about the correction need of the function cv::merge.

for example i made bold where i think it might be corrected:

Creates one multi-channel array out of several single-channel ones.

The function merge merges several arrays to make a single multi-channel array. That is, each element of the output array will be a concatenation of the elements of the input arrays, where elements of i-th input array are treated as mv[i].channels()-element vectors

The function cv::split does the reverse operation. If you need to shuffle channels in some other advanced way, use cv::mixChannels

Parameters

mv input array of matrices to be merged; all the matrices in mv must have the same size and the same depth.

count number of input matrices when mv is a plain C array; it must be greater than zero.

dst output array of the same size and the same depth as mv[0]; The number of channels will be the total number of channels in the matrix array. ( i think The number of channels will be given value by count)

here you can find the header file to be corrected

edit retag flag offensive reopen merge delete

Closed for the following reason the question is answered, right answer was accepted by sturkmen
close date 2017-09-21 02:39:22.502244

Comments

2

+1 for encouraging inexperienced users to do initial PR :).

Balaji R gravatar imageBalaji R ( 2015-11-02 09:51:00 -0500 )edit
1

On behalf of the inexperienced users: what's "PR"?

Guyygarty gravatar imageGuyygarty ( 2015-11-04 14:56:30 -0500 )edit
1
sturkmen gravatar imagesturkmen ( 2015-11-04 15:09:03 -0500 )edit
2

Hmm I do encourage the idea, but I am guessing most people interested in this are already looking through PR, topics and so on and help out where they can ;)

StevenPuttemans gravatar imageStevenPuttemans ( 2015-11-23 08:17:59 -0500 )edit

i do encourage someone to do initial PR again. otherwise i will do. i think the next version of OpenCV will be released soon!

sturkmen gravatar imagesturkmen ( 2015-12-14 07:44:21 -0500 )edit
1
sturkmen gravatar imagesturkmen ( 2015-12-15 12:42:41 -0500 )edit