Opened 6 years ago

Closed 6 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, GitHub, GitLab) Commit: bdcce24badfb4944792c817679c9025765a6f76d
Dependencies: Stopgaps:

Status badges

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 6 years ago by tmonteil

  • Branch set to u/tmonteil/explicit_in_the_documentation_that_elliptic_curves_are_plotted_over_the_reals_

comment:2 Changed 6 years ago by tmonteil

  • Commit set to bdcce24badfb4944792c817679c9025765a6f76d
  • Status changed from new to needs_review

New commits:

​bdcce24#17256 : documentation.

comment:3 Changed 6 years ago by chapoton

You should add a reference to the ticket using :trac:

comment:4 Changed 6 years ago by tmonteil

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 6 years ago by chapoton

  • Authors set to Thierry Monteil
  • 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 6 years ago by slelievre

can not -> cannot

comment:7 Changed 6 years ago by vbraun

  • 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
Note: See TracTickets for help on using tickets.