Ticket #7649 (new enhancement)

Opened 3 years ago

Last modified 3 years ago

improve documentation (etc) of totallyreal_rel

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

Description (last modified by was) (diff)

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

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

Change History

Changed 3 years ago by was

comment:1 Changed 3 years ago by was

  • Description modified (diff)

comment:2 Changed 3 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.

Note: See TracTickets for help on using tickets.