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

Update run scripts and environment files for NASA discover cluster #2439

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Aug 28, 2024

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

This PR updates the operational run scripts and environment files for using GC-Classic and GCHP on the NASA discover cluster. This includes environment files using both GNU and Intel fortran compiler on Milan nodes. GEOS-Chem CTM users on discover can easily access these files in GEOS-Chem run directories starting in version 14.5 via symlink runScriptSamples/operational_examples/nasa_discover.

Tagging GEOS-Chem users on discover: @ktravis213, @viral211, @aschuh

Expected changes

This is a zero diff update.

Reference(s)

None

Related Github Issue

None

@lizziel lizziel added topic: User Environment Relating to libraries, containers, AMIs, etc. no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations labels Aug 28, 2024
@lizziel lizziel added this to the 14.5.0 milestone Aug 28, 2024
@lizziel lizziel changed the base branch from main to dev/14.5.0 August 28, 2024 20:30
@yantosca yantosca self-requested a review August 28, 2024 20:34
@yantosca yantosca self-assigned this Aug 28, 2024
@yantosca yantosca added the category: Feature Request New feature or request label Aug 28, 2024
@lizziel lizziel force-pushed the feature/update_scripts_for_discover_cluster branch from c259b0c to 113f634 Compare August 28, 2024 20:35
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.

Hi @lizziel: This looks good to merge. I just had one minor suggestion (feel free to ignore it).

Copy link
Contributor

Thanks @lizziel. I'll pull this into alpha.8.

@yantosca
Copy link
Contributor

This PR was integration-tested with PR #2371 and PR #2436. All GEOS-Chem Classic integration tests passed:

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

CodeDir   : 153a355 GEOS-Chem submod update: Merge PR #2428 (vdiff) and PR #2144 (drydep)
GEOS-Chem : d34c567a4 Merge PR #2436 (Update GFED4 climatology for 2010-2023)
HEMCO     : d0d5fca PR #285 post-merge fix: Update CHANGELOG.md
Cloud-J   : d20050f Update version to 7.7.3
HETP      : 2a99b24 Merge pull request #2 from geoschem/bugfix/initialize_local_variables

Using 24 OpenMP threads
Number of execution tests: 30

Submitted as SLURM job: 45504789
==============================================================================

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

@yantosca
Copy link
Contributor

This PR was integration-tested with PR #2371 and PR #2436. All GCHP Classic integration tests passed:

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

CodeDir       : da8ea12 GEOS-Chem submod update: Merge PR #2428 (vdiff) and PR #2144 (drydep)
MAPL          : 231d53c Merge pull request #36 from geoschem/feature/improve_hflux_regridding
GMAO_Shared   : 4ddb3ec Merge pull request #2 from geoschem/feature/mapl-upgrade
ESMA_cmake    : ad5deba Added ecbuild as a submodule of ESMA_cmake
gFTL-shared   : 4b82492 Merge branch 'upstream_v1.5.0' into feature/v1.5.0
FMS           : 259759d Merge pull request #3 from geoschem/feature/update_gmao_libs
FVdycoreCubed : af42462 Merge PR #8 (Add PLEadv diagnostic for offline advection in GCHP)
geos-chem     : d34c567a4 Merge PR #2436 (Update GFED4 climatology for 2010-2023)
HEMCO         : d0d5fca PR #285 post-merge fix: Update CHANGELOG.md
yaFyaml       : 19afe50 Merge branch 'upstream_v1.0.4' into feature/v1.0.4
pFlogger      : 2c4b724 Merge branch 'upstream_v1.9.1' into feature/v1.9.1
Cloud-J       : d20050f Update version to 7.7.3
HETP          : 2a99b24 Merge pull request #2 from geoschem/bugfix/initialize_local_variables

Number of execution tests: 12

Submitted as SLURM job: 45515705
==============================================================================

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

@yantosca yantosca merged commit d4c47ba into dev/14.5.0 Aug 29, 2024
@yantosca yantosca deleted the feature/update_scripts_for_discover_cluster branch August 29, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Feature Request New feature or request no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations topic: User Environment Relating to libraries, containers, AMIs, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants