With Check

Author: manishvenu

visualCaseGen/custom_widget_types/case_creator.py

@@ -11,7 +11,7 @@
 from ProConPy.config_var import cvars
 from visualCaseGen.custom_widget_types.mom6_bathy_launcher import MOM6BathyLauncher
 from visualCaseGen.custom_widget_types.dummy_output import DummyOutput
-from visualCaseGen.custom_widget_types.case_tools import xmlchange, run_case_setup, append_user_nl
+from visualCaseGen.custom_widget_types.case_tools import xmlchange, run_case_setup, append_user_nl, is_ccs_config_writeable
 
 COMMENT = "\033[01;96m"  # bold, cyan
 SUCCESS = "\033[1;32m"  # bold, green
@@ -23,7 +23,7 @@
 class CaseCreator:
     """The base class for CaseCreatorWidget. Here, backend functionalities are implemented."""
 
-    def __init__(self, cime, output=None, allow_xml_override=False, ccs_config_read_only = False):
+    def __init__(self, cime, output=None, allow_xml_override=False):
         """Initialize CaseCreator object.
 
         Parameters
@@ -40,7 +40,7 @@ def __init__(self, cime, output=None, allow_xml_override=False, ccs_config_read_
         self._cime = cime
         self._out = DummyOutput() if output is None else output
         self._allow_xml_override = allow_xml_override
-        self._assign_grids_through_ccs_config = not ccs_config_read_only # By default, visualCaseGen assigns grids through ccs_config, if not possible (which is possible if the user does not own the sandbox), it applies the change through xml case changes.
+        self._assign_grids_through_ccs_config = is_ccs_config_writeable(cime) # By default, visualCaseGen assigns grids through ccs_config, if not possible (which can happen if the user does not own the sandbox), it applies the change through xml case changes.
 
     def revert_launch(self, do_exec=True):
         """This function is called when the case creation fails. It reverts the changes made
@@ -248,7 +248,7 @@ def _update_modelgrid_aliases(self, custom_grid_path, ocn_grid, do_exec):
         modelgrid_aliases_xml = modelgrid_aliases_xml.as_posix()
 
         # confirm that modelgrid_aliases xml file is writeable:
-        if not os.access(modelgrid_aliases_xml, os.W_OK):
+        if not is_ccs_config_writeable(self.cime):
             raise RuntimeError(f"Cannot write to {modelgrid_aliases_xml}.")
 
         # Construct the component grids string to be logged:

visualCaseGen/custom_widget_types/case_tools.py

@@ -8,6 +8,15 @@
 RESET = "\033[0m"
 
 
+def is_ccs_config_writeable(cime):
+    srcroot = cime.srcroot
+    ccs_config_root = Path(srcroot) / "ccs_config"
+    assert (
+        ccs_config_root.exists()
+    ), f"ccs_config_root {ccs_config_root} does not exist."
+    modelgrid_aliases_xml = ccs_config_root / "modelgrid_aliases_nuopc.xml"
+    return os.access(modelgrid_aliases_xml, os.W_OK)
+
 def run_case_setup(do_exec, is_non_local=False, out=None):
     """Run the case.setup script to set up the case instance.