Ticket #12184 (closed defect: duplicate)
Change example in doc/bordeaux/elliptic_curves.rst
| Reported by: | cremona | Owned by: | mvngu |
|---|---|---|---|
| Priority: | minor | Milestone: | sage-duplicate/invalid/wontfix |
| Component: | documentation | Keywords: | rd2 |
| Cc: | wstein | Work issues: | |
| Report Upstream: | N/A | Reviewers: | R. Andrew Ohana |
| Authors: | Merged in: | ||
| Dependencies: | Stopgaps: |
Description (last modified by cremona) (diff)
In William Stein's Bordeaux lectures he gave an example of an elliptic curve *not in the tables* (of conductor 154907). But as of version 4.8 that curve is in the tables in the optional larger database, and so that example causes a doctest failure in sage/doc/en/bordeaux/elliptic_curves.rst.
The patch changes the example to one whose conductor is larger.
To test: apply the patch and test sage/doc/en/bordeaux/elliptic_curves.rst both before and after installing the optional package at #11587 (i.e. http://wstein.org/home/ohanar/cremona-database/database_cremona_ellcurve-20111029.spkg).
Attachments
-
trac_12184-ex.patch
(919 bytes) -
added by cremona 17 months ago.
- Apply to 4.8 (e.g. 4.8.alpha4)
Change History
Changed 17 months ago by cremona
-
attachment
trac_12184-ex.patch
added
comment:1 Changed 17 months ago by cremona
- Status changed from new to needs_review
- Description modified (diff)
comment:2 Changed 14 months ago by jen
- Milestone changed from sage-5.0 to sage-duplicate/invalid/wontfix
This is a near-duplicate of #12617, so we should close this ticket in favor of that one.
Note: See
TracTickets for help on using
tickets.
Apply to 4.8 (e.g. 4.8.alpha4)