geoschem / HEMCO

The Harmonized Emissions Component (HEMCO), developed by the GEOS-Chem Support Team.
https://hemco.readthedocs.io
Other
15 stars 31 forks source link

Restrict volcano climatology prints to root thread #263

Closed lizziel closed 3 months ago

lizziel commented 3 months ago

Name and Institution (Required)

Name: Christoph Keller Institution: GMAO

Describe the update

This PR is a bug fix that limits the warning message about using climatological volcano emissions to the root CPU.

Expected changes

This is a no diff update.

Reference(s)

None

Related Github Issue(s)

https://github.com/GEOS-ESM/HEMCO/pull/13

yantosca commented 3 months ago

Integration tests are running

yantosca commented 3 months ago
==============================================================================
GCHP: Execution Test Results

GCHP      #9bcf9f3 HEMCO submod update: Merge PR #262 (Increase nested brackets)
GEOS-Chem #67220bb52 Merge PR #2178 (Rename "native" to "raw" in GCHP rundir scripts)
HEMCO     #22de25e Merge PR #263 (Restrict volcano clim prints to root thread)

Number of execution tests: 6

Submitted as SLURM job: 23327606
==============================================================================

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

Also, all GCHP integration tests were zero-diff w/r/t 14.3.0

yantosca commented 3 months ago

All GCClassic integration tests passed:

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

GCClassic #bd16d4a HEMCO submod update: Merge PR #262 (Increase nested brackets)
GEOS-Chem #67220bb52 Merge PR #2178 (Rename "native" to "raw" in GCHP rundir scripts)
HEMCO     #22de25e Merge PR #263 (Restrict volcano clim prints to root thread)

Using 24 OpenMP threads
Number of execution tests: 26

Submitted as SLURM job: 23336212
==============================================================================

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

Also, all integration tests were zero-diff w/r/t 14.3.0, except:

yantosca commented 3 months ago

NOTE: This was merged into the dev/no-diff-to-benchmark branch.