From bd89e33e765d73aa2b48419031d6922b8bf5eb37 Mon Sep 17 00:00:00 2001 From: "Robert C. Helling" Date: Sun, 19 Feb 2017 10:28:55 +0100 Subject: Fix typo in saturation/desatureation rates This patch eliminates the difference between the saturation and desaturation rates. This was probably once meant as a conservative measure but the desaturation rate was increased rather than the saturation rate (which is probably a typo, as reported by Stefan). Since there is no good basis for this anyway, this patch sets both factors to 1.0 (and if accepted the whole factor business should be removed). This makes our deco times slightly longer. But in the past, we had introduced a 1.2% fudge factor in the critical radius calculation to add conservatism and match the benchmark better. Removing this fudge factor brings us close to the benchmarks. Expected test values updated. Reported-by: Stefan Signed-off-by: Robert C. Helling --- core/deco.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'core') diff --git a/core/deco.c b/core/deco.c index 5014f1b61..8247800da 100644 --- a/core/deco.c +++ b/core/deco.c @@ -27,7 +27,7 @@ // Subsurface appears to produce marginally less conservative plans than our benchmarks // Introduce 1.2% additional conservatism -#define subsurface_conservatism_factor 1.012 +#define subsurface_conservatism_factor 1.0 extern bool in_planner(); @@ -48,7 +48,7 @@ struct buehlmann_config { struct buehlmann_config buehlmann_config = { .satmult = 1.0, - .desatmult = 1.01, + .desatmult = 1.0, .last_deco_stop_in_mtr = 0, .gf_high = 0.75, .gf_low = 0.35, -- cgit v1.2.3-70-g09d2