From c0d25249df32bee28bcd5d80cbef416c09e3ae80 Mon Sep 17 00:00:00 2001 From: David A Minton Date: Wed, 30 Nov 2022 09:29:49 -0500 Subject: [PATCH] Fixed issue where units were being recomputed even if they hadn't changed --- python/swiftest/swiftest/simulation_class.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/python/swiftest/swiftest/simulation_class.py b/python/swiftest/swiftest/simulation_class.py index aeedc8ac6..d5c6e21b8 100644 --- a/python/swiftest/swiftest/simulation_class.py +++ b/python/swiftest/swiftest/simulation_class.py @@ -1772,7 +1772,10 @@ def set_unit_system(self, if all(key in self.param for key in ["MU2KG","DU2M","TU2S"]): self.GU = constants.GC * self.param["TU2S"] ** 2 * self.param["MU2KG"] / self.param["DU2M"] ** 3 - if recompute_unit_values: + if recompute_unit_values and \ + MU2KG_old != self.param['MU2KG'] or \ + DU2M_old != self.param['DU2M'] or \ + TU2S_old != self.param['TU2S']: self.update_param_units(MU2KG_old, DU2M_old, TU2S_old) unit_dict = self.get_unit_system(update_list, verbose) @@ -1868,7 +1871,6 @@ def update_param_units(self, MU2KG_old, DU2M_old, TU2S_old): if MU2KG_old is not None: for k in mass_keys: if k in self.param: - print(f"param['{k}']: {self.param[k]}") self.param[k] *= MU2KG_old / self.param['MU2KG'] if DU2M_old is not None: