Opened 12 years ago

Closed 11 years ago

#10292 closed defect (fixed)

padics, precision, list() and printing

Reported by: David Roe Owned by: David Roe
Priority: minor Milestone: sage-4.8
Component: padics Keywords:
Cc: Merged in: sage-4.8.alpha4
Authors: David Roe Reviewers: Johan Bosman
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Johan Bosman)

sage: E = EllipticCurve('37a')
sage: R = E.padic_regulator(7)
sage: len(R.list())
37
sage: s = str(R)
sage: len(R.list())
19

Similarly:

sage: E = EllipticCurve('37a')
sage: R = E.padic_regulator(7)
sage: len(R.list())
37
sage: R.precision_absolute()
20
sage: len(R.list())
37
sage: R.precision_relative()
19
sage: len(R.list())
19

This is likely due to _normalize not getting called when necessary at object creation.

Apply:

  1. 10292.patch
  2. 10292_review.patch

to the sage repository.

Attachments (2)

10292.patch (3.3 KB) - added by David Roe 11 years ago.
10292_review.patch (1.0 KB) - added by Johan Bosman 11 years ago.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by David Roe

Attachment: 10292.patch added

comment:1 Changed 11 years ago by David Roe

Status: newneeds_review

Changed 11 years ago by Johan Bosman

Attachment: 10292_review.patch added

comment:2 Changed 11 years ago by Johan Bosman

Authors: David Roe
Description: modified (diff)
Reviewers: Johan Bosman

I've uploaded a patch that fixes the docstring formatting. If you agree with this, this can be given a positive_review.

comment:3 Changed 11 years ago by David Roe

Status: needs_reviewpositive_review

Looks good to me. I should look over the Sphinx documentation to see when .. is advisable. The lack of :: was just an oversight.

comment:4 Changed 11 years ago by Jeroen Demeyer

Merged in: sage-4.8.alpha4
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.