Opened 13 years ago

Last modified 5 weeks ago

#7649 new enhancement

improve documentation (etc) of totallyreal_rel

Reported by: was Owned by: was
Priority: minor Milestone:
Component: number theory Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by was)

Here is a (small) patch which includes some explanation and references in the docstrings for totallyreal_rel.py.

The roundoff error is a bit mystifying to me, but in any case I
believe I fixed it in the patch: just expanding the bounds works, and seems to only slow it down by a small amount.

In truth, I think the issue has to do with the fact that math.floor
and math.ceil return float64 not ints, and so I have to coerce these
ints, but I couldn't verify this.


 -- John Voight

Attachments (1)

tr_doc.patch (4.2 KB) - added by was 13 years ago.

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by was

Attachment: tr_doc.patch added

comment:1 Changed 13 years ago by was

Description: modified (diff)

comment:2 Changed 13 years ago by kcrisman

Just a couple comments on non-mathematical things... I think there is now a new reference system with .. , as Minh has recently noted in the developer guide, and "Parallelepiped" seems to be the most common spelling and is the one in my dictionary, though see the Wikipedia article for other, less common, spellings.

comment:3 Changed 9 years ago by jdemeyer

Milestone: sage-5.11sage-5.12

comment:4 Changed 9 years ago by vbraun_spam

Milestone: sage-6.1sage-6.2

comment:5 Changed 9 years ago by vbraun_spam

Milestone: sage-6.2sage-6.3

comment:6 Changed 8 years ago by vbraun_spam

Milestone: sage-6.3sage-6.4

comment:7 Changed 5 weeks ago by mkoeppe

Milestone: sage-6.4
Note: See TracTickets for help on using tickets.