Synfig Issue Tracker
star_faded.png
Please log in to bookmark issues
enhancement_small.png
CLOSED  Enhancement #324  -  Add new convert type: Angle -> Real
Posted Jan 09, 2013 - updated Jan 05, 2019
action_vote_minus_faded.png
0
Votes
 action_vote_plus_faded.png
icon_info.png This issue has been closed with status "Testing / QA" and resolution "RESOLVED".
Issue details
  • Type of issue
    Enhancement
  • Status
     
    		 Testing / QA
  • Assigned to
    Konstantin Dmitriev
  • Progress
       
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
  • Posted by
    Konstantin Dmitriev
  • Owned by
    Not owned by anyone
  • Time spent
    1 week, 6 hours
  • Category
    Not determined
  • Resolution
    RESOLVED
  • Priority
    Not determined
  • Targetted for
    icon_milestones.png 0.64.0
  • Tags
    icon_customdatatype.png Not determined
  • Difficulty
    icon_customdatatype.png Easy
Issue description
Please add a new convert type which allows to convert Angle values to Real.

The convert type should be available for reverse manipulations.

#3
Comment posted by
Carlos López
Jan 10, 06:51
Meanwhile it is coded, you can obtain the Angle from Real from the current convert types:

First convert your Angle to Scale. It would give you two parameters: Link (angle) and Scale (real). Placing a proper value to the Link sub parameter would allow you to obtain angles from reals (the Scale). For example to have degrees by real number just fill Link with 1.0º. To obtain radians from reals just fill Link with 57,2957795º. Unfortunately the angle value type doesn't allow more than two decimals and that's the bug that must be fixed.

Anyway, there would be more convert types for angles from real based on its types: degrees, turns and radians. Also the trigonometric convert types aren't complete: I miss ACos and ASin (Atan2 is already coded).
#4
Comment posted by
Konstantin Dmitriev
icon_reply.pngJan 10, 17:19, in reply to comment #3
Yes, there is a workaround, but it won't be available for reverse manipulations.

I believe it won't be a problem to take existing Integer convert type and create the similar for Real convert.
#8
Comment posted by
Konstantin Dmitriev
Jan 11, 18:42
Implemented in morevna branch. See pull request here - https://github.com/synfig/synfig/pull/4
#12
Comment posted by
Carlos López
Mar 30, 15:41
Applied to master branch

The issue was updated with the following change(s):
  • This issue has been closed
  • The resolution has been updated, from Not determined to RESOLVED.
  • Information about the user working on this issue has been changed, from Konstantin Dmitriev to Not being worked on.
  • Time spent on this issue, from No time spent to 1 week, 6 hours.