Skip to content

Commit

Permalink
don't unconditionally increase length bound
Browse files Browse the repository at this point in the history
fixes #109
  • Loading branch information
malb committed Apr 23, 2024
1 parent e49c32b commit 60808bd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion estimator/sis_lattice.py
Expand Up @@ -203,7 +203,9 @@ def cost_zeta(
"""
# step 0. establish baseline cost using worst case euclidean norm estimate
# length_bound =1 makes sense when norm=∞, but we take logs and divide
params_baseline = params.updated(norm=2, length_bound=params.length_bound + 1)
params_baseline = params.updated(
norm=2, length_bound=2 if params.length_bound == 1 else params.length_bound
)
baseline_cost = lattice(
params_baseline,
ignore_qary=ignore_qary,
Expand Down

0 comments on commit 60808bd

Please sign in to comment.