Opened 5 years ago
Closed 5 years ago
#17256 closed defect (fixed)
Explicit in the documentation that elliptic curves are plotted over the reals.
Reported by: | tmonteil | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | documentation | Keywords: | |
Cc: | Merged in: | ||
Authors: | Thierry Monteil | Reviewers: | Frédéric Chapoton |
Report Upstream: | N/A | Work issues: | |
Branch: | bdcce24 (Commits) | Commit: | bdcce24badfb4944792c817679c9025765a6f76d |
Dependencies: | Stopgaps: |
Description
On this ask question it has been reported that plotting an elliptic curve defined on a quadratic. Actually the plot method works on RR
(if there is a natural coercion from the base ring of the curve to RR
).
Change History (7)
comment:1 Changed 5 years ago by
- Branch set to u/tmonteil/explicit_in_the_documentation_that_elliptic_curves_are_plotted_over_the_reals_
comment:2 Changed 5 years ago by
- Commit set to bdcce24badfb4944792c817679c9025765a6f76d
- Status changed from new to needs_review
comment:3 Changed 5 years ago by
You should add a reference to the ticket using :trac:
comment:4 Changed 5 years ago by
I just tried something like:
An elliptic curve defined over the Complex Field can not be plotted (see :trac:`17256`):: sage: E = EllipticCurve(CC, [0,0,1,-1,0]) sage: E.plot() Traceback (most recent call last): ... NotImplementedError: Plotting of curves over Complex Field with 53 bits of precision not implemented yet
but i am not convinced it helps in understanding in any way, since this ticket does not fixes any particular bug, just notices an existing fact. The link to this trac ticket looks misleading. Do you have another way in mind ?
comment:5 Changed 5 years ago by
- Reviewers set to Frédéric Chapoton
- Status changed from needs_review to positive_review
ok, I agree that the reference to trac is not very useful here. Good to go
comment:6 Changed 5 years ago by
can not -> cannot
comment:7 Changed 5 years ago by
- Branch changed from u/tmonteil/explicit_in_the_documentation_that_elliptic_curves_are_plotted_over_the_reals_ to bdcce24badfb4944792c817679c9025765a6f76d
- Resolution set to fixed
- Status changed from positive_review to closed
New commits:
#17256 : documentation.