Ticket #3645 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch; positive review] tutorial: make documentation for .n() more prominent

Reported by: jhpalmieri Owned by: tba
Priority: minor Milestone: sage-3.1
Component: documentation Keywords: tutorial, editor_mhansen
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

In response to the discussion in the thread

 http://groups.google.com/group/sage-support/browse_frm/thread/3c6972e69d1b80d8

I've tried to make the documentation for the .n() method more prominent.

Attachments

3645.patch Download (2.3 KB) - added by jhpalmieri 5 years ago.
doc-3645-part2.patch Download (1.4 KB) - added by was 5 years ago.
3645-newest.patch Download (2.4 KB) - added by jhpalmieri 5 years ago.

Change History

comment:1 Changed 5 years ago by jhpalmieri

  • Summary changed from tutorial: make documentation for .n() more prominent to [with patch; needs review] tutorial: make documentation for .n() more prominent

(Oh, and I made one other little change: in the notebook interface, the documentation link is called "Help", not "Documentation", so I changed the tutorial to reflect this.)

Changed 5 years ago by jhpalmieri

comment:2 Changed 5 years ago by cremona

Looks good to me!

Changed 5 years ago by was

comment:3 Changed 5 years ago by was

  • Summary changed from [with patch; needs review] tutorial: make documentation for .n() more prominent to [with patch; positive review] tutorial: make documentation for .n() more prominent

REFEREE REPORT:

I've added a patch that

  1. Note that capital N is a synomym for n?
  2. Note that digits is the number of *decimal* digits?

With that I give this a positive review.

-- William

comment:4 Changed 5 years ago by jhpalmieri

One small change: I changed William's new example to use N(-), since we didn't have an example like that yet.

This new patch replaces all of the other patches (I thought that would be easier than having to apply three patches).

Changed 5 years ago by jhpalmieri

comment:5 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from sage-3.1.1 to sage-3.1

Merged 3645-newest.patch in Sage 3.1.alpha0

Note: See TracTickets for help on using tickets.