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

 
 

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

Submitter:  Doug Stewart <dastew>
Submitted:  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
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group 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>
Group 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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group 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.14-f13d.
    Corresponding source code