Hi everyone,

I hope I am posting at the correct place.

There is a minor bug in the verbose output of the small_roots function in
Z/nZ[X].
At line 567 (Sage 9.2, I might not be absolutely up to date…) of the
build/pkgs/sagelib/src/build/lib.linux-x86_64-3.8/sage/rings/polynomial/polynomial_modn_dense_ntl.pyx
file, the variable epsilon is output as a integer when it is actually a
float
567:    verbose("epsilon = %d"%epsilon, level=2)
I attach a minimal working example which outputs variable epsilon as 0 when
it is indeed strictly positive.
I would be glad to contribute if you indicate me the procedure to follow,
or someone more powerful than me could just change the line to
567:    verbose("epsilon = %f"%epsilon, level=2)
and push on the repo?

Thank you very much for all your work,

Adam

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sage-devel/CA%2BXQvf1RHTsLqwt%2B6heVAZjhk2%3Dxj2MO2OCZcQ88sEzhGMGQ4A%40mail.gmail.com.
#!/usr/bin/env python3
# -*- coding: utf-8 -*-
"""
Created on Wed Nov  4 09:32:22 2020

@author: larat
"""


from sage.all_cmdline import *   # import sage library
R1x    = PolynomialRing(Integers(2**16+1), names=('x',));(x,) = R1x._first_ngens(1)
P = x**3-27
set_verbose(2)
smallRoots = P.small_roots(X=10,beta=1.0,epsilon=0.1)
print(smallRoots[0])

Reply via email to