Ticket #3645 (closed enhancement: fixed)

Opened 5 months ago

Last modified 4 months ago

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

Reported by: jhpalmieri Assigned to: tba
Priority: minor Milestone: sage-3.1
Component: documentation Keywords: tutorial, editor_mhansen
Cc:

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 (2.3 kB) - added by jhpalmieri on 07/11/2008 03:06:22 PM.
doc-3645-part2.patch (1.4 kB) - added by was on 07/24/2008 03:35:49 AM.
3645-newest.patch (2.4 kB) - added by jhpalmieri on 07/24/2008 07:20:44 PM.

Change History

07/11/2008 03:01:39 PM changed 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.)

07/11/2008 03:06:22 PM changed by jhpalmieri

  • attachment 3645.patch added.

07/11/2008 03:07:02 PM changed by cremona

Looks good to me!

07/24/2008 03:35:49 AM changed by was

  • attachment doc-3645-part2.patch added.

07/24/2008 03:36:00 AM changed 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

07/24/2008 07:20:20 PM changed 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).

07/24/2008 07:20:44 PM changed by jhpalmieri

  • attachment 3645-newest.patch added.

07/30/2008 04:21:27 PM changed 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