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: |
Description (last modified by )
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:
to the sage repository.
Attachments (2)
Change History (6)
Changed 11 years ago by
Attachment: | 10292.patch added |
---|
comment:1 Changed 11 years ago by
Status: | new → needs_review |
---|
Changed 11 years ago by
Attachment: | 10292_review.patch added |
---|
comment:2 Changed 11 years ago by
Authors: | → David Roe |
---|---|
Description: | modified (diff) |
Reviewers: | → Johan Bosman |
comment:3 Changed 11 years ago by
Status: | needs_review → positive_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
Merged in: | → sage-4.8.alpha4 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
I've uploaded a patch that fixes the docstring formatting. If you agree with this, this can be given a positive_review.