Ok Andrey, Thank you for the guidelines. I will proceed with the modifications. I will let you know if I need any help. 

wrt C-wrapper, I had asked Vadim before my pr, and he told me to keep it. 

----- Original Message -----

> From: "Andrey Kamaev" <notifications@github.com>
> To: "Itseez/opencv" <opencv@noreply.github.com>
> Cc: "evangelidis" <georgios.evangelidis@inria.fr>
> Sent: Tuesday, February 12, 2013 3:31:22 PM
> Subject: Re: [opencv] ecc algorithm (#459)

> In modules/video/doc/motion_analysis_and_object_tracking.rst: > @@
> -153,6 +153,50 @@ In case of point sets, the problem is formulated
> as follows: you need to find a
> >      :ocv:func:`getPerspectiveTransform`,
> >      :ocv:func:`findHomography`
> >
> > +findTransformECC
> > +------------------------
> > +Finds the geometric transform (warp) between two images in terms
> > of the ECC criterion [EP08]_.
> > +
> > +.. ocv:function:: double findTransformECC( InputArray
> > templateImage, InputArray inputImage, InputOutputArray warpMatrix,
> > int motionType=2, TermCriteria
> > criteria=TermCriteria(TermCriteria::COUNT+TermCriteria::EPS, 50,
> > 0.001))
> > +.. ocv:cfunction:: double cvFindTransformECC( const CvArr*
> > templateImage, const CvArr* inputImage, CvArr* warpMatrix, const
> > int motionType, CvTermCriteria criteria)
> The 4th parameter is named motionType here but in header it is named
> as warpECC . It makes a problem for automatic parsers so
> declarations should be unified. Please change the declaration in
> tracking.hpp to use the same names for all arguments.
> —
> Reply to this email directly or view it on GitHub .