diff --git a/config/cesm/machines/config_machines.xml b/config/cesm/machines/config_machines.xml index 287e5152267..0bf72bf1c2f 100644 --- a/config/cesm/machines/config_machines.xml +++ b/config/cesm/machines/config_machines.xml @@ -996,7 +996,7 @@ This allows using a different mpirun command to launch unit tests NCAR AMD EPYC de.*.hpc.ucar.edu CNL - intel + intel,gnu mpich $ENV{SCRATCH} diff --git a/scripts/fortran_unit_testing/run_tests.py b/scripts/fortran_unit_testing/run_tests.py index dab5da396fe..63efa914e0d 100755 --- a/scripts/fortran_unit_testing/run_tests.py +++ b/scripts/fortran_unit_testing/run_tests.py @@ -130,7 +130,7 @@ def parse_command_line(args): args = CIME.utils.parse_args_and_handle_standard_logging_options(args, parser) output = Printer(color=args.color) - + if args.xml_test_list is None and args.test_spec_dir is None: output.print_error( "You must specify either --test-spec-dir or --xml-test-list." diff --git a/scripts/lib/CIME/SystemTests/system_tests_common.py b/scripts/lib/CIME/SystemTests/system_tests_common.py index 0a38c49ad02..67a6ec629cd 100644 --- a/scripts/lib/CIME/SystemTests/system_tests_common.py +++ b/scripts/lib/CIME/SystemTests/system_tests_common.py @@ -399,6 +399,7 @@ def _compare_baseline(self): """ compare the current test output to a baseline result """ + memlist = None with self._test_status: # compare baseline success, comments = compare_baseline(self._case) diff --git a/scripts/lib/CIME/XML/env_batch.py b/scripts/lib/CIME/XML/env_batch.py index 80e75db041a..687973415f5 100644 --- a/scripts/lib/CIME/XML/env_batch.py +++ b/scripts/lib/CIME/XML/env_batch.py @@ -328,7 +328,8 @@ def get_batch_directives(self, case, job, overrides=None, output_format='default directive_prefix = None roots = self.get_children("batch_system") - queue = self.get_value("JOB_QUEUE", subgroup=job) + # This must go through the case object + queue = case.get_value("JOB_QUEUE", subgroup=job) if self._batchtype != "none" and not queue in self._get_all_queue_names(): unknown_queue = True qnode = self.get_default_queue() diff --git a/scripts/lib/CIME/XML/machines.py b/scripts/lib/CIME/XML/machines.py index dc42579515e..5c2f9c33ff3 100644 --- a/scripts/lib/CIME/XML/machines.py +++ b/scripts/lib/CIME/XML/machines.py @@ -252,7 +252,7 @@ def is_valid_compiler(self,compiler): """ Check the compiler is valid for the current machine - >>> machobj = Machines(machine="cori-haswell") + >>> machobj = Machines(machine="derecho") >>> machobj.get_default_compiler() 'intel' >>> machobj.is_valid_compiler("gnu") @@ -266,7 +266,7 @@ def is_valid_MPIlib(self, mpilib, attributes=None): """ Check the MPILIB is valid for the current machine - >>> machobj = Machines(machine="cori-haswell") + >>> machobj = Machines(machine="derecho") >>> machobj.is_valid_MPIlib("mpi-serial") True >>> machobj.is_valid_MPIlib("fake-mpi") @@ -279,7 +279,7 @@ def has_batch_system(self): """ Return if this machine has a batch system - >>> machobj = Machines(machine="cori-haswell") + >>> machobj = Machines(machine="derecho") >>> machobj.has_batch_system() True >>> machobj.set_machine("melvin") diff --git a/scripts/lib/CIME/code_checker.py b/scripts/lib/CIME/code_checker.py index e25b6ce163e..6c3c23570b0 100644 --- a/scripts/lib/CIME/code_checker.py +++ b/scripts/lib/CIME/code_checker.py @@ -15,7 +15,7 @@ ############################################################################### def _run_pylint(on_file, interactive): ############################################################################### - pylint = find_executable("pylint") + pylint = find_executable("pylint") cmd_options = " --disable=I,C,R,logging-not-lazy,wildcard-import,unused-wildcard-import" cmd_options += ",fixme,broad-except,bare-except,eval-used,exec-used,global-statement" diff --git a/scripts/tests/scripts_regression_tests.py b/scripts/tests/scripts_regression_tests.py index b155106e09b..1a921ac7177 100755 --- a/scripts/tests/scripts_regression_tests.py +++ b/scripts/tests/scripts_regression_tests.py @@ -2764,7 +2764,7 @@ def check_for_pylint(): from distutils.spawn import find_executable pylint = find_executable("pylint") if pylint is not None: - output = run_cmd_no_fail("pylint --version") + output = run_cmd_no_fail(pylint+" --version") pylintver = re.search(r"pylint\s+(\d+)[.](\d+)[.](\d+)", output) major = int(pylintver.group(1)) minor = int(pylintver.group(2))