Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explicit in the documentation that elliptic curves are plotted over the reals. #17256

Closed
sagetrac-tmonteil mannequin opened this issue Oct 29, 2014 · 10 comments
Closed

Comments

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Oct 29, 2014

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).

Component: documentation

Author: Thierry Monteil

Branch/Commit: bdcce24

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/17256

@sagetrac-tmonteil sagetrac-tmonteil mannequin added this to the sage-6.4 milestone Oct 29, 2014
@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Nov 4, 2014

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Nov 4, 2014

Commit: bdcce24

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Nov 4, 2014

New commits:

bdcce24#17256 : documentation.

@fchapoton
Copy link
Contributor

comment:3

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

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Nov 4, 2014

comment:4

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 ?

@fchapoton
Copy link
Contributor

comment:5

ok, I agree that the reference to trac is not very useful here. Good to go

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

Author: Thierry Monteil

@slel
Copy link
Member

slel commented Nov 13, 2014

comment:6

can not -> cannot

@vbraun
Copy link
Member

vbraun commented Nov 15, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants