PyImageSearch / imutils

A series of convenience functions to make basic image processing operations such as translation, rotation, resizing, skeletonization, and displaying Matplotlib images easier with OpenCV and Python.
MIT License
4.54k stars 1.03k forks source link

angle in rotate() and rotate_bound() has opposite meaning. wtf? #205

Open nkrot opened 4 years ago

nkrot commented 4 years ago

Both functions -- rotate() and rotate_bound() -- have a parameter for specifying the angle of rotation. Surprisingly, the meaning of it is different:

  1. rotate(image, 10) will rotate the image 10 deg counterclockwise (-10)
  2. rotate_bound(image, 10) will rotate the image 10 deg clockwise (+10)

What was the motivation for such a design decision? Would it make sense to unify the interpretation of the angle parameter? Or at least add a note to (barely existing) documentation?

Thanks, Nikolai