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

Add Cloud-J as submodule to GCHP #356

Merged
merged 8 commits into from
Dec 12, 2023
Merged

Add Cloud-J as submodule to GCHP #356

merged 8 commits into from
Dec 12, 2023

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Nov 13, 2023

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

This update adds the Cloud-J photolysis library as a submodule with GCHP. It is stored alongside GEOS-Chem in src/GCHP_GridComp/Chem_GridComp. This PR should be merged with geoschem/geos-chem#1522.

Expected changes

See geoschem/geos-chem#1522 for changes expected in GEOS-Chem when using Cloud-J to compute J-values.

Reference(s)

Prather, M. J.: Photolysis rates in correlated overlapping cloud fields: Cloud-J 7.3c, Geosci. Model Dev., 8, 2587–2595, https://doi.org/10.5194/gmd-8-2587-2015, 2015.

Related Github Issue(s)

geoschem/geos-chem#1522

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Default is currently off (will use Fast-JX)

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
…TJX=y

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
  > Remove unused common variables JXL_, JXL1_, and JXL2_
  > Make number of CTM levels set by the calling program during initialization
  > Merge pull request #4 from jimmielin/hplin/fix_decl_ifort
  > Updates for using MAPL in Cloud-J

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel changed the base branch from dev/14.3.0 to main November 29, 2023 15:14
@lizziel lizziel requested a review from msulprizio November 29, 2023 15:14
@lizziel lizziel changed the title [WIP] Add Cloud-J as submodule to GCHP Add Cloud-J as submodule to GCHP Nov 29, 2023
@lizziel lizziel requested a review from yantosca November 29, 2023 15:16
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.

Thanks @lizziel. This looks good to merge.

@yantosca yantosca added category: Feature Request New feature or request topic: Build Related to makefiles or the build sequence external modules labels Nov 29, 2023
@yantosca yantosca added this to the 14.3.0 milestone Nov 29, 2023
# Conflicts:
#	CHANGELOG.md

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@msulprizio msulprizio changed the base branch from main to dev/14.3.0 December 12, 2023 16:57
@msulprizio msulprizio merged commit df9d5c2 into dev/14.3.0 Dec 12, 2023
@msulprizio msulprizio deleted the feature/cloud-j branch December 12, 2023 19:43
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 topic: Build Related to makefiles or the build sequence
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants