geoschem / HEMCO

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

Limit GC_72_EDGE_SIGMA variable to 135 columns per Fortran-90 spec #234

Closed jimmielin closed 1 year ago

jimmielin commented 1 year ago

This will make compile compatible with the nvhpc compiler.

This was reported downstream from ESCOMP/CAM#871 and at HEMCO #233.

Reported-by: Jian Sun @sjsprecious

Name and Institution (Required)

Name: Haipeng Lin Institution: Harvard U.

Confirm you have reviewed the following documentation

Describe the update

Use continuation marks for GC_72_EDGE_SIGMA variable specific to MODEL_WRF and MODEL_CESM.

This will make it compatible with Fortran spec, as nvhpc compiler will throw an error about this line of code.

Expected changes

Zero diff, pure reformatting of code lines.

Reference(s)

N/A

Related Github Issue(s)

jimmielin commented 1 year ago

Done, thanks @yantosca!

yantosca commented 1 year ago

This PR was opened atop PR https://github.com/geoschem/geos-chem/pull/1895 (which was the previous integration test).

All GEOS-Chem Classic integration tests passed, except for TOMAS, which is a known issue that will be fixed later.

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

GCClassic #c1eb4a2 GEOS-Chem submod update: Merge PR #1985 (Impose BC's at every timestep)
GEOS-Chem #e1e675266 Update info in integration test/parallel test README.md files
HEMCO     #c874025 Add corresponding CHANGELOG.md update

Using 24 OpenMP threads
Number of execution tests: 26

Submitted as SLURM job: 65707631
==============================================================================

Execution tests:
------------------------------------------------------------------------------
gc_05x0625_NA_47L_merra2_CH4........................Execute Simulation....PASS
gc_05x0625_NA_47L_merra2_fullchem...................Execute Simulation....PASS
gc_4x5_47L_merra2_fullchem..........................Execute Simulation....PASS
gc_4x5_47L_merra2_fullchem_TOMAS15..................Execute Simulation....FAIL
gc_4x5_47L_merra2_fullchem_TOMAS40..................Execute Simulation....FAIL
gc_4x5_merra2_aerosol...............................Execute Simulation....PASS
gc_4x5_merra2_carbon................................Execute Simulation....PASS
gc_4x5_merra2_CH4...................................Execute Simulation....PASS
gc_4x5_merra2_CO2...................................Execute Simulation....PASS
gc_4x5_merra2_fullchem..............................Execute Simulation....PASS
gc_4x5_merra2_fullchem_aciduptake...................Execute Simulation....PASS
gc_4x5_merra2_fullchem_APM..........................Execute Simulation....PASS
gc_4x5_merra2_fullchem_benchmark....................Execute Simulation....PASS
gc_4x5_merra2_fullchem_complexSOA...................Execute Simulation....PASS
gc_4x5_merra2_fullchem_complexSOA_SVPOA.............Execute Simulation....PASS
gc_4x5_merra2_fullchem_LuoWd........................Execute Simulation....PASS
gc_4x5_merra2_fullchem_marinePOA....................Execute Simulation....PASS
gc_4x5_merra2_fullchem_RRTMG........................Execute Simulation....PASS
gc_4x5_merra2_Hg....................................Execute Simulation....PASS
gc_4x5_merra2_metals................................Execute Simulation....PASS
gc_4x5_merra2_POPs_BaP..............................Execute Simulation....PASS
gc_4x5_merra2_tagCH4................................Execute Simulation....PASS
gc_4x5_merra2_tagCO.................................Execute Simulation....PASS
gc_4x5_merra2_tagO3.................................Execute Simulation....PASS
gc_4x5_merra2_TransportTracers......................Execute Simulation....PASS
gc_4x5_merra2_TransportTracers_LuoWd................Execute Simulation....PASS

Summary of test results:
------------------------------------------------------------------------------
Execution tests passed: 24
Execution tests failed: 2
Execution tests not yet completed: 0

Furthermore, all integration tests were zero-diff w/r/t PR #1895, except for:

Checking gc_05x0625_NA_47L_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem_TOMAS15 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem_TOMAS40 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_aerosol -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_carbon -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_CH4 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_CO2 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_aciduptake -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_APM -> 2 differences found in OutputDir

Checking gc_4x5_merra2_fullchem_benchmark -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_complexSOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_complexSOA_SVPOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_LuoWd -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_marinePOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_RRTMG -> 1 difference found in OutputDir

Checking gc_4x5_merra2_Hg -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_metals -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_POPs_BaP -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagCH4 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagCO -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagO3 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_TransportTracers -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_TransportTracers_LuoWd -> No differences in OutputDir -> No differences in Restarts

yantosca commented 1 year ago

This PR was opened atop PR https://github.com/geoschem/geos-chem/pull/1895 (which was the previous integration test).

All GEOS-Chem Classic integration tests passed, except for TOMAS, which is a known issue that will be fixed later.

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

GCClassic #c1eb4a2 GEOS-Chem submod update: Merge PR #1985 (Impose BC's at every timestep)
GEOS-Chem #e1e675266 Update info in integration test/parallel test README.md files
HEMCO     #c874025 Add corresponding CHANGELOG.md update

Using 24 OpenMP threads
Number of execution tests: 26

Submitted as SLURM job: 65707631
==============================================================================

Execution tests:
------------------------------------------------------------------------------
gc_05x0625_NA_47L_merra2_CH4........................Execute Simulation....PASS
gc_05x0625_NA_47L_merra2_fullchem...................Execute Simulation....PASS
gc_4x5_47L_merra2_fullchem..........................Execute Simulation....PASS
gc_4x5_47L_merra2_fullchem_TOMAS15..................Execute Simulation....FAIL
gc_4x5_47L_merra2_fullchem_TOMAS40..................Execute Simulation....FAIL
gc_4x5_merra2_aerosol...............................Execute Simulation....PASS
gc_4x5_merra2_carbon................................Execute Simulation....PASS
gc_4x5_merra2_CH4...................................Execute Simulation....PASS
gc_4x5_merra2_CO2...................................Execute Simulation....PASS
gc_4x5_merra2_fullchem..............................Execute Simulation....PASS
gc_4x5_merra2_fullchem_aciduptake...................Execute Simulation....PASS
gc_4x5_merra2_fullchem_APM..........................Execute Simulation....PASS
gc_4x5_merra2_fullchem_benchmark....................Execute Simulation....PASS
gc_4x5_merra2_fullchem_complexSOA...................Execute Simulation....PASS
gc_4x5_merra2_fullchem_complexSOA_SVPOA.............Execute Simulation....PASS
gc_4x5_merra2_fullchem_LuoWd........................Execute Simulation....PASS
gc_4x5_merra2_fullchem_marinePOA....................Execute Simulation....PASS
gc_4x5_merra2_fullchem_RRTMG........................Execute Simulation....PASS
gc_4x5_merra2_Hg....................................Execute Simulation....PASS
gc_4x5_merra2_metals................................Execute Simulation....PASS
gc_4x5_merra2_POPs_BaP..............................Execute Simulation....PASS
gc_4x5_merra2_tagCH4................................Execute Simulation....PASS
gc_4x5_merra2_tagCO.................................Execute Simulation....PASS
gc_4x5_merra2_tagO3.................................Execute Simulation....PASS
gc_4x5_merra2_TransportTracers......................Execute Simulation....PASS
gc_4x5_merra2_TransportTracers_LuoWd................Execute Simulation....PASS

Summary of test results:
------------------------------------------------------------------------------
Execution tests passed: 24
Execution tests failed: 2
Execution tests not yet completed: 0

Furthermore, all integration tests were zero-diff w/r/t PR https://github.com/geoschem/geos-chem/pull/1895, except for:

Checking gc_05x0625_NA_47L_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem_TOMAS15 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_47L_merra2_fullchem_TOMAS40 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_aerosol -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_carbon -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_CH4 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_CO2 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_aciduptake -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_APM -> 2 differences found in OutputDir

Checking gc_4x5_merra2_fullchem_benchmark -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_complexSOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_complexSOA_SVPOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_LuoWd -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_marinePOA -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_fullchem_RRTMG -> 1 difference found in OutputDir

Checking gc_4x5_merra2_Hg -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_metals -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_POPs_BaP -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagCH4 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagCO -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_tagO3 -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_TransportTracers -> No differences in OutputDir -> No differences in Restarts

Checking gc_4x5_merra2_TransportTracers_LuoWd -> No differences in OutputDir -> No differences in Restarts

lizziel commented 1 year ago

We will need to bring this update into a Z version of HEMCO 3.6 so that it can be used ASAP and with GEOS-Chem 14.1.1. I can make a PR for this.