github/concrete
1
1
Fork 0
mirror of https://github.com/zama-ai/concrete.git synced 2026-02-08 19:44:57 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: Don't let the optimizer to be to optimistic, up to 16 bits we didn't have yet a solution (#131)

Browse Source
This commit is contained in:
Quentin Bourgerie
2022-08-12 10:32:18 +02:00
committed by GitHub
parent 226559e48d
commit 4fffc26bbc
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
concrete-optimizer/src/optimization/dag/solo_key/optimize_generic.rs
View File

@@ -10,6 +10,7 @@ use crate::optimization::wop_atomic_pattern::optimize::optimize_one as wop_optim
use crate::optimization::wop_atomic_pattern::Solution as WopSolution;
const MINIMAL_WOP_PRECISION: Precision = 9;
const MAXIMAL_WOP_PRECISION: Precision = 16;
pub enum Solution {
WpSolution(WpSolution),
@@ -54,6 +55,8 @@ pub fn optimize<W: UnsignedInteger>(
)
.best_solution;
opt_sol.map(Solution::WpSolution)
} else if max_precision > MAXIMAL_WOP_PRECISION {
None
} else {
let fallback_16b_precision = 16;
let default_log_norm = default_log_norm2_woppbs;