bugGNU Octave - Bugs: bug #57052, Polar should document that input...

 
 

bug #57052: Polar should document that input is in radians

Submitted by:  Doug Stewart <dastew>
Submitted on:  Sun 13 Oct 2019 06:36:57 PM UTC  
 
Category:  Documentation Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Documentation
Status:  Fixed Assigned to:  None
Originator Name:  Doug Stewart Open/Closed:  Closed
Release:  5.1.0 Operating System:  Any

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Tue 15 Oct 2019 12:34:39 AM UTC, comment #6: 

I cleaned up the patch and added a seealso link to cart2pol and checked it in here https://hg.savannah.gnu.org/hgweb/octave/rev/eeb2275424bc.

Thanks for the patch.  Marking as fixed and closing report.

Rik <rik5>
Project Administrator
Mon 14 Oct 2019 04:41:49 PM UTC, comment #5: 

ok it should be there now. sorry.

(file #47667)

Doug Stewart <dastew>
Mon 14 Oct 2019 04:39:34 PM UTC, comment #4: 

Could you check the upload?  I don't see any file attached to the bug report.

Rik <rik5>
Project Administrator
Mon 14 Oct 2019 02:53:30 PM UTC, comment #3: 

I attached a first attempt at better documentation.

Doug Stewart <dastew>
Mon 14 Oct 2019 02:32:54 PM UTC, comment #2: 

I will work on it.

Doug Stewart <dastew>
Sun 13 Oct 2019 10:29:37 PM UTC, comment #1: 

See the documentation from Matlab (https://www.mathworks.com/help/matlab/ref/polar.html).  This is the expected result, input in radians with labeling in degrees. 

Can you prepare a patch for the documentation to mention that the input is in radians?

Rik <rik5>
Project Administrator
Sun 13 Oct 2019 06:36:57 PM UTC, original submission:  

When using the polar command to plot, it draws an axis with 0 - 360 deg shown. The documentation also uses degrees. But it is assuming that the input data is in radians.

q=rand(1,100);
c=ones(1,numel(q))*2; # a vector of 2s
polar(c,q,'.')

This should draw a line at 2 degrees but it is a 2 radians.
We can change the documentation or the calculations.

Doug Stewart <dastew>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #47667:  27478.patch added by dastew (1002B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by dastew (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-10-15 rik5 StatusConfirmed => Fixed
        Open/ClosedOpen => Closed
    2019-10-14 dastew Attached File- => Added 27478.patch, #47667
    2019-10-13 rik5 CategoryPlotting => Documentation
        Item GroupIncorrect Result => Documentation
        StatusNone => Confirmed
        SummaryPolar uses radians but the polar axis is in deg. => Polar should document that input is in radians

    Back to the top


    Powered by Savane 3.5