After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 669365 - rotate plugin docs say that 'angle' is in radians, yet seems to be in degrees
rotate plugin docs say that 'angle' is in radians, yet seems to be in degrees
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal normal
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-02-04 15:28 UTC by Rory McCann
Modified: 2012-02-09 18:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rotate: fix description string to match what the element does (942 bytes, patch)
2012-02-04 22:07 UTC, Vincent Penquerc'h
committed Details | Review
rotate: angle is in radians already, do not scale it (998 bytes, patch)
2012-02-04 22:07 UTC, Vincent Penquerc'h
committed Details | Review

Description Rory McCann 2012-02-04 15:28:41 UTC
The documentation (via gst-insepct) for the rotate plugin in -bad says that the angle paramters is in radians, however when i use it it seems to be in degrees.

Firstly angle=45 produces a good 4͏5° rotated angle, and angle=30 shows the video rotated at 30͏°. angle=0.78 (0.78 radians is approximately 45͏°) shows the video only rotated by a little small amount, not 45͏°.

I am using version 0.10.22-2ubuntu4 from ubuntu.

Either the documentation or the code should be changed.
Comment 1 Petteri Aimonen 2012-02-04 16:06:54 UTC
I see the same, and also the description is wrong:

  Description:	Warps the picture into an arc shaped form

Looks like the rotate plugin was based off the circle plugin and someone forgot to update the documentation strings.
Comment 2 Vincent Penquerc'h 2012-02-04 22:07:55 UTC
Created attachment 206784 [details] [review]
rotate: fix description string to match what the element does
Comment 3 Vincent Penquerc'h 2012-02-04 22:07:58 UTC
Created attachment 206785 [details] [review]
rotate: angle is in radians already, do not scale it

Other plugins use radians, and the angle documentation says radians.
Comment 4 Vincent Penquerc'h 2012-02-05 20:33:20 UTC
commit ddff93320dda8460c22fbe3cc0567b40fb4a0f31
Author: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date:   Sat Feb 4 22:06:57 2012 +0000

    rotate: angle is in radians already, do not scale it
    
    Other plugins use radians, and the angle documentation says radians.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=669365

commit e84bbcbeabece21849f45d51b6e0f7b766cfa4dd
Author: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date:   Sat Feb 4 22:06:12 2012 +0000

    rotate: fix description string to match what the element does
    
    https://bugzilla.gnome.org/show_bug.cgi?id=669365