From 273ddaa8082195b79cd62bd5b341eb2a67485a6d Mon Sep 17 00:00:00 2001 From: jakehader Date: Mon, 3 Oct 2022 15:32:34 -0700 Subject: [PATCH] Add settings validator for checking if `detailedAxialExpansion` and `nonUniformAssemFlags` are provided together. If so, recommend removing the `nonUniformAssemFlags` settings and setting it back to its default --- armi/physics/neutronics/settings.py | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/armi/physics/neutronics/settings.py b/armi/physics/neutronics/settings.py index a85590340..bff0a0785 100644 --- a/armi/physics/neutronics/settings.py +++ b/armi/physics/neutronics/settings.py @@ -24,6 +24,10 @@ ) from armi.settings import setting from armi.utils import directoryChangers +from armi.settings.fwSettings.globalSettings import ( + CONF_DETAILED_AXIAL_EXPANSION, + CONF_NON_UNIFORM_ASSEM_FLAGS, +) CONF_BC_COEFFICIENT = "bcCoefficient" @@ -509,4 +513,17 @@ def migrateDpaGridPlate(): ) ) + queries.append( + settingsValidation.Query( + lambda: inspector.cs[CONF_DETAILED_AXIAL_EXPANSION] + and inspector.cs[CONF_NON_UNIFORM_ASSEM_FLAGS], + f"The use of {CONF_DETAILED_AXIAL_EXPANSION} and {CONF_NON_UNIFORM_ASSEM_FLAGS} is not supported.", + "Automatically set non-uniform assembly treatment to its default?", + lambda: inspector._assignCS( + CONF_NON_UNIFORM_ASSEM_FLAGS, + inspector.cs.getSetting(CONF_NON_UNIFORM_ASSEM_FLAGS).default, + ), + ) + ) + return queries