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

Basic implementation of point counting using canonical lift for elliptic curve. #11448

Closed
jpflori opened this issue Jun 8, 2011 · 10 comments
Closed

Comments

@jpflori
Copy link

jpflori commented Jun 8, 2011

The proposed patch implements a basic version of point counting for elliptic curve using canonical lift (à la Satoh).

This implements the algorithms described in Fouquet, Gaudry and Harley, "An extension of Satoh's algorithm and its implementation", http://hal.inria.fr/inria-00512791/en, based on the Pari/GP implementation by Yeoh, http://pages.cs.wisc.edu/~yeoh/nt/satoh-fgh.gp.

It uses Pari for computation in Z_q.

This is currently only implemented for characteristic two.

Other characteristics are nearly done, but I have some bugs left.

It adds a cardinality_fgh() method to the EllipticCurve_finite_field class and the real implementation is made in a new fgh_algo.py file.

CC: @jpflori @mminzlaff @defeo @zimmermann6

Component: elliptic curves

Keywords: point counting, satoh, canonical lift

Work Issues: Fix F_8

Author: Jean-Pierre Flori

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

@jpflori
Copy link
Author

jpflori commented Jun 8, 2011

Attachment: trac_11448-fgh_pari.patch.gz

Basic implementation.

@jpflori

This comment has been minimized.

@jpflori
Copy link
Author

jpflori commented Jun 8, 2011

comment:2

I just realized there is a bug when working over F_8.

I'll try to fix it quickly.

@jpflori
Copy link
Author

jpflori commented Jun 8, 2011

Work Issues: Fix F_8

@roed314
Copy link
Contributor

roed314 commented Oct 15, 2012

comment:5

Any progress on this?

@jpflori
Copy link
Author

jpflori commented Oct 15, 2012

comment:6

PARI now has very efficient point counting in small characteristic (faster than MAGMA), we should wrap these functionnalities.
I have as well code based on FLINT yielding similar performance as PARI.

I've put some timings of the PARI code (originally posted on pari-dev list) at #11548.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@kedlaya
Copy link
Sponsor Contributor

kedlaya commented Aug 17, 2016

comment:11

There is a ticket about exposing the PARI implementation at #16931.

@jpflori
Copy link
Author

jpflori commented Aug 17, 2016

comment:12

This is way out of date and should be closed!

@jpflori jpflori removed this from the sage-6.4 milestone Aug 17, 2016
@jpflori
Copy link
Author

jpflori commented Aug 17, 2016

comment:13

Superseded by #16931.

@embray
Copy link
Contributor

embray commented Aug 30, 2016

comment:14

Determined to be invalid/duplicate/wontfix (closing as "wontfix" as a catch-all resolution).

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

6 participants