From 6b29e165992e26c1e8df73d01febcec4e5623ea2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexandre=20P=C3=A9r=C3=A9?= Date: Thu, 9 Jan 2025 09:51:49 +0100 Subject: [PATCH] fix(ci): add missing fontconfig dependency --- .github/workflows/concrete_optimizer.yml | 5 +++++ .../src/optimization/dag/multi_parameters/optimize/mod.rs | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/concrete_optimizer.yml b/.github/workflows/concrete_optimizer.yml index 01e79d067..9aa1427a3 100644 --- a/.github/workflows/concrete_optimizer.yml +++ b/.github/workflows/concrete_optimizer.yml @@ -42,6 +42,11 @@ jobs: cargo --version cd compilers/concrete-optimizer cargo fmt --check + - name: Fix fontconfig on ubuntu + if: ${{ matrix.runson == 'ubuntu-latest' }} + run: | + sudo apt update + sudo apt -y install libfontconfig1-dev - name: Build run: | cd compilers/concrete-optimizer diff --git a/compilers/concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs b/compilers/concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs index 0b8cdf9b1..d68d7a64b 100644 --- a/compilers/concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs +++ b/compilers/concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs @@ -536,7 +536,7 @@ fn apply_fks_variance_and_cost_or_lower_bound( // if an optimized fks is applicable and is not to be optimized // we use the already optimized fks instead of a lower bound if let Some(this_fks) = initial_fks[src.0][dst.0] { - let to_be_optimized = fks_to_optimize[src.0].map_or(false, |fdst| dst == fdst); + let to_be_optimized = fks_to_optimize[src.0] == Some(dst); if !to_be_optimized { if input_glwe == &this_fks.src_glwe_param && output_glwe == &this_fks.dst_glwe_param {