Difference in documentation of ellipse and ellipse2poly (Bug #3301)

Added by Steven Puttemans over 11 years ago. Updated over 11 years ago.

Status:Done Start date:2013-10-03
Priority:Normal Due date:
Assignee:Steven Puttemans % Done:

50%
Category:documentation
Target version:2.4.7
Affected version:2.4.6 (latest release) Operating System:Windows
Difficulty:Easy HW Platform:x64
Pull request:https://github.com/Itseez/opencv/pull/1538

Description

Looking at the documentation of ellipse, it is said to use the axes size, but the ellipse2poly mentions to use half of the axes size. I think that both should have the same description in order not to confuse people.

If you could suggest which approach is best, then I will happily supply a pullrequest.

Personally, I think that the ellipse function should mention that you need half of the axes size, in stead of the size of the axes.

http://docs.opencv.org/modules/core/doc/drawing_functions.html?highlight=ellipse#ellipse

Associated revisions

Revision 2f474db2
Added by Steven Puttemans over 11 years ago

Fix of bug #3283 and bug #3301
Added extra information for fitEllipse, minAreaRect and addapted ellipse &
ellipse2Poly parameter description.

Revision 6fa6e3d3
Added by Vadim Pisarevsky over 10 years ago

Merge pull request #3301 from MC-Escherichia:fix_gen_javadoc_py_prints

History

#1
Updated by Anna Kogan over 11 years ago

  • Status changed from New to Open

#2
Updated by Steven Puttemans over 11 years ago

Will fix this problem in another pull request I made : https://github.com/Itseez/opencv/pull/1538

  • % Done changed from 0 to 50

#3
Updated by Kirill Kornyakov over 11 years ago

Thanks! Fix going to be merged soon...

  • Pull request set to https://github.com/Itseez/opencv/pull/1538
  • Status changed from Open to Done

Also available in: Atom PDF