From b1c01b25c8f2c3947be23f5b2c656c37437dab17 Mon Sep 17 00:00:00 2001 From: Deepak Mallubhotla Date: Sun, 19 May 2024 22:08:44 -0500 Subject: [PATCH] fix: Adds ugly hack for stdevs for this uniform range to multiply by root3, proper fix would be in pdme --- deepdog/subset_simulation/subset_simulation_impl.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/deepdog/subset_simulation/subset_simulation_impl.py b/deepdog/subset_simulation/subset_simulation_impl.py index 9344645..3e23669 100644 --- a/deepdog/subset_simulation/subset_simulation_impl.py +++ b/deepdog/subset_simulation/subset_simulation_impl.py @@ -84,10 +84,16 @@ class SubsetSimulation: self.mcmc_seed = mcmc_seed self.use_adaptive_steps = use_adaptive_steps - self.default_phi_step = default_phi_step + self.default_phi_step = ( + default_phi_step * 1.73 + ) # this is a hack to fix a missing sqrt 3 in the proposal function code. self.default_theta_step = default_theta_step - self.default_r_step = default_r_step - self.default_w_log_step = default_w_log_step + self.default_r_step = ( + default_r_step * 1.73 + ) # this is a hack to fix a missing sqrt 3 in the proposal function code. + self.default_w_log_step = ( + default_w_log_step * 1.73 + ) # this is a hack to fix a missing sqrt 3 in the proposal function code. self.default_upper_w_log_step = default_upper_w_log_step _logger.info("using params:")