Skip to content

Commit

Permalink
undo the changes to the hist_utils move tool and remove suffix in ssp…
Browse files Browse the repository at this point in the history
… test (no hist files produced)
  • Loading branch information
jedwards4b committed Sep 2, 2016
1 parent 7262448 commit 66a5b48
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 12 deletions.
2 changes: 1 addition & 1 deletion utils/python/CIME/SystemTests/ssp.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ def run_phase(self):
clone.flush()

dout_sr = clone.get_value("DOUT_S_ROOT")
self.run_indv(suffix="spinup", st_archive=True)
self.run_indv(suffix="", st_archive=True)

#-------------------------------------------------------------------
# (2) do a hybrid, non-spinup run in orig_case
Expand Down
6 changes: 1 addition & 5 deletions utils/python/CIME/SystemTests/system_tests_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -223,12 +223,8 @@ def _coupler_log_indicates_run_complete(self):
return False

def _component_compare_move(self, suffix):
success, comments = move(self._case, suffix)
comments = move(self._case, suffix)
append_status(comments, sfile="TestStatus.log")
status = TEST_PASS_STATUS if success else TEST_FAIL_STATUS
with self._test_status:
self._test_status.set_status("%s_%s" % (COMPARE_PHASE, suffix), status)
return success

def _component_compare_test(self, suffix1, suffix2):
"""
Expand Down
9 changes: 3 additions & 6 deletions utils/python/CIME/hist_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,12 +75,9 @@ def move(case, suffix):
comments += " Copying '%s' to '%s'\n" % (test_hist, new_file)
shutil.copy(test_hist, new_file)

all_success = True
if num_moved == 0:
all_success = False
comments += "WARNING: No hist files found in rundir '%s'" % rundir
expect(num_moved > 0, "move failed: no hist files found in rundir '%s'" % rundir)

return all_success, comments
return comments


def _hists_match(model, hists1, hists2, suffix1="", suffix2=""):
Expand Down Expand Up @@ -312,12 +309,12 @@ def get_extension(model, filepath):

m = ext_regex.match(basename)
expect(m is not None, "Failed to get extension for file '%s'" % filepath)

if m.group(1) is not None:
result = m.group(1)+'.'+m.group(2)
else:
result = m.group(2)


return result

def generate_baseline(case, baseline_dir=None, allow_baseline_overwrite=False):
Expand Down

0 comments on commit 66a5b48

Please sign in to comment.