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

issue in documentation of small_roots #14815

Open
zimmermann6 opened this issue Jun 25, 2013 · 1 comment
Open

issue in documentation of small_roots #14815

zimmermann6 opened this issue Jun 25, 2013 · 1 comment

Comments

@zimmermann6
Copy link

the documentation of small_roots says:

   Let N be the characteristic of the base ring this polynomial is
   defined over: "N = self.base_ring().characteristic()". This method
   returns small roots of this polynomial modulo some factor b of N
   with the constraint that b >= N^beta. Small in this context means
   that if x is a root of f modulo b then |x| < X. This X is either
   provided by the user or the maximum X is chosen such that this
   algorithm terminates in polynomial time. If X is chosen
   automatically it is X = ceil(1/2 N^{beta^2/delta - epsilon}).

However the parameter delta is not defined.
Looking at the source code it seems it is the degree of the input polynomial,
but this should be confirmed and documented.

CC: @robertwb

Component: documentation

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

@zimmermann6
Copy link
Author

comment:1

Robert, since you are the only author of polynomial_modn_dense_ntl.pyx, I add you in cc.
Please can you confirm that delta is the degree of the input polynomial?

Paul

@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
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
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

2 participants