Merge pull request #105 from malb/issue104

fix issue 104
malb · Apr 8, 2024 · 7ea215a · 7ea215a
2 parents 00ec72c + fc5e63f
commit 7ea215a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/estimator/reduction.py b/estimator/reduction.py
@@ -382,7 +382,9 @@ def _short_vectors_sieve(self, beta, d, N=None, B=None, preprocess=True, sieve_d
         elif N is None:
             N = floor(2 ** (0.2075 * beta))  # pick something
 
-        c = N / floor(2 ** (0.2075 * beta))
+        c0 = RR(N)
+        c1 = RR(2 ** (RR(0.2075 * beta)))
+        c = c0 / c1
 
         rho = sqrt(4 / 3.0) * RR(
             self.delta(sieve_dim) ** (sieve_dim - 1) * self.delta(beta) ** (1 - sieve_dim)
@@ -391,7 +393,7 @@ def _short_vectors_sieve(self, beta, d, N=None, B=None, preprocess=True, sieve_d
         return (
             rho,
             ceil(c) * self(beta, d),
-            ceil(c) * floor(2 ** (0.2075 * beta)),
+            ceil(c) * floor(c1),
             sieve_dim,
         )