Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rename HEMCO_Config.rc.sample to HEMCO_Config.rc #245

Merged
merged 1 commit into from
Nov 20, 2023
Merged

rename HEMCO_Config.rc.sample to HEMCO_Config.rc #245

merged 1 commit into from
Nov 20, 2023

Conversation

nicholasbalasus
Copy link
Contributor

Name and Institution (Required)

Name: Nick Balasus
Institution: Harvard University

Confirm you have reviewed the following documentation

Describe the update

In createRunDir.sh, users are given the option to type ./HEMCO_Config.rc.sample if they do not have a HEMCO_Config.rc file. Currently, HEMCO_Config.rc.sample is copied directly without changing the name, meaning this code does not run successfully because HEMCO_Config.rc does not exist.

HEMCO/run/createRunDir.sh

Lines 366 to 368 in 00aaf65

sed_ie "s|{DATA_ROOT}|${GC_DATA_ROOT}|" HEMCO_Config.rc
sed_ie "s|{GRID_DIR}|${grid_dir}|" HEMCO_Config.rc
sed_ie "s|{MET_DIR}|${met_dir}|" HEMCO_Config.rc

To fix this, I rename HEMCO_Config.rc.sample to HEMCO_Config.rc. This is consistent with what is done for HEMCO_Diagn.rc.

cp ./HEMCO_Diagn.rc.sample ${rundir}/HEMCO_Diagn.rc

Expected changes

Only to HEMCO standalone run directory creations where the user specifies ./HEMCO_Config.rc.sample.

Reference(s)

n/a

Related Github Issue(s)

n/a

@yantosca yantosca changed the base branch from dev/3.8.0 to dev/no-diff-to-benchmark November 20, 2023 18:59
@yantosca yantosca self-requested a review November 20, 2023 19:49
@yantosca yantosca self-assigned this Nov 20, 2023
@yantosca yantosca added topic: Configuration Files Related to HEMCO configuration files category: Bug Fix Fixes a bug that was previously reported topic: HEMCO Standalone Model Pertaining to the HEMCO standalone no-diff-to-benchmark This update will have no impact on benchmark simulations labels Nov 20, 2023
@yantosca yantosca added this to the 3.7.2 milestone Nov 20, 2023
Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to merge @nicholasbalasus. We can get this into the dev/no-diff-to-benchmark branch, for the upcoming GC 14.2.3 / HEMCO 3.7.2 patch release.

@yantosca
Copy link
Contributor

All GEOS-Chem Classic integration tests (except TOMAS) passed:

==============================================================================
GEOS-Chem Classic: Execution Test Results

GCClassic #f5b5694 Merge branch 'docs/dev' into dev/14.2.2
GEOS-Chem #382f11dca Merge PR #2040 (Lower GCHP timestep threshold if > C180)
HEMCO     #6ab11ea Merge PR #245 (Rename HEMCO_Config.rc.sample to HEMCO_Config.rc)

Using 24 OpenMP threads
Number of execution tests: 27

Submitted as SLURM job: 10285010
==============================================================================

Summary of test results:
------------------------------------------------------------------------------
Execution tests passed: 25
Execution tests failed: 2                # <==== TOMAS15 and TOMAS40
Execution tests not yet completed: 0

All GEOS-Chem Classic integration tests were zero-diff w/r/t 14.2.2, except TOMAS (failed), APM (parallelization issues) and RRTMG (parallelization issue in diagnostic output).

All GCHP integration tests passed:

==============================================================================
GCHP: Execution Test Results

gchp      #074494e GEOS-Chem submodule update: Merge PR #2009 fixing links to RTD
GEOS-Chem #382f11dca Merge PR #2040 (Lower GCHP timestep threshold if > C180)
HEMCO     #6ab11ea Merge PR #245 (Rename HEMCO_Config.rc.sample to HEMCO_Config.rc)

Number of execution tests: 5

Submitted as SLURM job: 10285450
==============================================================================

%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%%  All execution tests passed!  %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

All GCHP integration tests were also zero-diff w/r/t 14.2.2.

@yantosca yantosca merged commit 6ab11ea into geoschem:dev/no-diff-to-benchmark Nov 20, 2023
1 check passed
yantosca added a commit that referenced this pull request Dec 1, 2023
This is the official release of HEMCO 3.7.2.  Updates include:

- Add utility scripts to change version numbers before release (PR #242)
- Rename HEMCO Config.rc.sample to HEMCO_Config.rc for standalone (PR #245)
- Increase string length for netCDF variable name (PR #248)
- Turn off emission extensions when EMISSIONS logical is false (PR #250)

Signed-off-by: Bob Yantosca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Fix Fixes a bug that was previously reported no-diff-to-benchmark This update will have no impact on benchmark simulations topic: Configuration Files Related to HEMCO configuration files topic: HEMCO Standalone Model Pertaining to the HEMCO standalone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants