Ticket #10367 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

plot3d transformation documentation says 'independent' when it should be 'dependent'

Reported by: jason Owned by: mvngu
Priority: minor Milestone: sage-4.6.1
Component: documentation Keywords: beginner
Cc: Work issues:
Report Upstream: N/A Reviewers: Jason Grout
Authors: Benjamin Jones Merged in: sage-4.6.1.rc0
Dependencies: Stopgaps:

Description

As pointed out on

 http://ask.sagemath.org/question/220/new-project-looking-for-help

the doctests for this function:

http://www.sagemath.org/doc/reference/sage/plot/plot3d/plot3d.html#sage.plot.plot3d.plot3d.Spherical

say:

"We can plot with this transform. Remember that the independent variable is the radius, and the dependent variables are the azimuth and the inclination (in that order):"

But it should read:

"We can plot with this transform. Remember that the dependent variable is the radius, and the independent variables are the azimuth and the inclination (in that order):"

Attachments

trac_10367_docfix.patch Download (1.5 KB) - added by benjaminfjones 2 years ago.
fix documentation error in Spherical and Cylindrical

Change History

comment:2 Changed 2 years ago by benjaminfjones

  • Status changed from new to needs_review
  • Authors set to Benjamin Jones

I noticed this too when working on Trac #10368. Patch is included. I checked that the doctests pass.

Changed 2 years ago by benjaminfjones

fix documentation error in Spherical and Cylindrical

comment:3 Changed 2 years ago by jason

  • Status changed from needs_review to positive_review
  • Reviewers set to Jason Grout

Looks great to me. Thanks!

comment:4 Changed 2 years ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.6.1.rc0
Note: See TracTickets for help on using tickets.