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

Updates for GEOS-Chem #2

Merged
merged 37 commits into from
Mar 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
c5d4a8a
Set different values for certain parameters if using GEOS-Chem Classic
lizziel Oct 26, 2021
80aa9de
Change GEOS-Chem compiler definition from GCCLASSIC to GEOSCHEM
lizziel Feb 27, 2023
cd90d6b
Add module variables BRANCH and RNAMES for more photolysis species info
lizziel Feb 27, 2023
e3d03b0
Pass tables data directory path when initializing Cloud-J
lizziel Feb 27, 2023
75df703
Update prints to reduce whitespace printed to log to improve readability
lizziel Feb 27, 2023
d290e28
Compute new global variables RNAMES and BRANCH for species info
lizziel Feb 27, 2023
9a3fd51
Explicitly declare subroutines in fjx_sub_mod as public or private
lizziel Feb 27, 2023
02f5d9d
List Cloud_JX declared args; remove CLDFLAG, LNRG, NRANDO since global
lizziel Feb 27, 2023
b32c4be
Rename Cloud-J global variable FP to FPAR to avoid conflict in GEOS-Chem
lizziel Feb 27, 2023
fe312d9
Bug fix: initialize REFFLX and REFFLIX for case of CLDFLAG = 1
lizziel Feb 27, 2023
b930d81
Change FJX_scat-aer.dat char width of RAAJ and DAAJ from 6 to 7
lizziel Feb 27, 2023
6ca5fa9
Change FJX_scat-aer.dat char width of QAA from 7 to 9, and PAA from 6…
lizziel Feb 27, 2023
766472c
First pass of reformatting files for readability
lizziel Feb 27, 2023
13b4b78
Minor updates, including fixing write statement format to fit # GC jv…
lizziel May 19, 2023
d052d2d
Fix bug in Cloud-J standalone for computing ice water path input to C…
lizziel Jun 2, 2023
1929de9
Remove unused inputs to ICA_* subroutines called in Cloud_JX
lizziel Jun 2, 2023
4d612ea
Update comments in cld_sub_mod for clarity
lizziel Jun 2, 2023
b1b7aec
Add clear sky option (compilte-time) for testing
lizziel Oct 17, 2023
8e68fee
Add debug logicals for turning on/off ice and liquid water cloud
lizziel Oct 17, 2023
0d83efa
Remove debugging on/off logicals from fjx_sub_mod.F90
lizziel Nov 13, 2023
b27c34b
Include compile definition for MODEL_GEOSCHEM when building in GCHP
lizziel Nov 14, 2023
96aa415
Updates for using MAPL in Cloud-J
lizziel Nov 14, 2023
9f0a5a6
Fix ANU declaration after use for ifort compatibility
jimmielin Nov 16, 2023
867a70c
Merge pull request #4 from jimmielin/hplin/fix_decl_ifort
lizziel Nov 17, 2023
cba1bcd
Make number of CTM levels set by the calling program during initializ…
lizziel Nov 28, 2023
a546892
Remove unused common variables JXL_, JXL1_, and JXL2_
lizziel Nov 28, 2023
365b488
Pass root thread logical to init subroutines to limit prints to 1 core
lizziel Jan 9, 2024
e683f37
Merge pull request #5 for branch 'feature/pass_am_I_root' into geos-chem
msulprizio Jan 10, 2024
aced055
Updates core files to all use cldj_ prefix and .F90 suffix
lizziel Feb 8, 2024
be7e5e7
Change CPP def ESMF_ to MAPL_ESMF; use MODEL_GEOSCHEM if GCHP or GC-C…
lizziel Feb 9, 2024
458d910
Merge pull request #6 from GEOS-ESM/geos/latest_gcc
lizziel Feb 15, 2024
9adf45d
Add GitHub config and PR/issue templates
lizziel Feb 15, 2024
18bc5ba
Remove redunction file close; update global var file for clarity
lizziel Feb 28, 2024
417a9ea
Updates for Cloud-J standalone for compatibility with GEOS-Chem changes
lizziel Mar 12, 2024
43afee3
Update standalone FJX_scat-aer.dat with format changes
lizziel Mar 12, 2024
6fca3e6
Add initialization fix to avoid error detected by gfortran
lizziel Mar 13, 2024
8921f2b
Add changelog
lizziel Mar 13, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
blank_issues_enabled: false
contact_links:
- name: Cloud-J README
url: https://github.com/geoschem/cloud-j
about: Please see the Cloud-J README and docs subdirectory on GitHub for more information
13 changes: 13 additions & 0 deletions .github/ISSUE_TEMPLATE/new-feature-or-discussion.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
---
name: Request a new Cloud-J feature or start a discussion
about: Use this form to request a new Cloud-J feature or start a discussion
---

### Name and Institution (Required)

Name:
Institution:

### New Cloud-J feature or discussion

Please provide a clear and concise overview of the discussion topic or new feature requested.
14 changes: 14 additions & 0 deletions .github/ISSUE_TEMPLATE/question-issue.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
---
name: Ask a question about Cloud-J or report an issue with Cloud-J
about: Use this form to ask a question about Cloud-J or to report an issue
---

### Name and Institution (Required)

Name:
Institution:

### Description of your issue or question

Please provide as much detail as possible. Always include the Cloud-J version number and any relevant configuration and log files.

20 changes: 20 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
### Name and Institution (Required)

Name:
Institution:

### Describe the update

Please provide a clear and concise overview of the update.

### Expected changes

Please provide details on how this update will impact model output and include plots or tables as needed.

### Reference(s)

If this is a science update, please provide a literature citation.

### Related Github Issues and PRs

Please link to the corresponding Github issue here. If fixing a bug, there should be an issue describing it with steps to reproduce.
13 changes: 13 additions & 0 deletions .github/no-response.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# Configuration for probot-no-response - https://github.com/probot/no-response

# Number of days of inactivity before an Issue is closed for lack of response
daysUntilClose: 60
# Label requiring a response
responseRequiredLabel: more information needed
# Comment to post when closing an Issue for lack of response. Set to `false` to disable
closeComment: >
This issue has been automatically closed because there has been no response
to our request for more information from the original author. With only the
information that is currently in the issue, we don't have enough information
to take action. Please reach out if you have or find the answers we need so
that we can investigate further.
20 changes: 20 additions & 0 deletions .github/stale.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# Number of days of inactivity before an issue becomes stale
daysUntilStale: 30
# Number of days of inactivity before a stale issue is closed
daysUntilClose: 7
# Issues with these labels will never be considered stale
exemptLabels:
- never stale
- feature
- discussion
- future development
# Label to use when marking an issue as stale
staleLabel: stale
# Comment to post when marking an issue as stale. Set to `false` to disable
markComment: >
This issue has been automatically marked as stale because it has not had
recent activity. If there are no updates within 7 days it will be closed.
You can add the "never stale" tag to prevent the Stale bot from closing
this issue.
# Comment to post when closing a stale issue. Set to `false` to disable
closeComment: Closing due to inactivity
48 changes: 48 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
# Changelog

This file documents all notable changes to the Cloud-J repository since the initial commit under git version control.

The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [7.7.1] - TBD
### Changed
- Changed arguments of Init_Cldj to include root thread logical and LUT data directory
- Renamed global variable FP to FPAR to avoid name conflict in other models
- Changed FJX_scat-aer.dat format to expect 1 character greater width for R-eff and rho
- Changed FJX_scat-aer.dat format to expect 2 characters greater width for QAA (data column 2)
- Changed FJX_scat-aer.dat format to expect 1 character greater width for PAA (data columns 3-9)
- Limit prints to single core
- Renamed all modules to use .F90 suffix and cldj_ prefix

## Added
- Added CMake support
- Added .gitignore, authors list, and contribution and support guidelines
- Added module for error handling
- Added C pre-processor blocks for compatibility with GEOS-Chem offline CTM, GCHP, GEOS, and CESM
- Added new global variables RNAMES and BRANCH for species info
- Added GitHub config and PR/issue templates
- Added GitHub action for build tests

### Fixed
- Fixed bugs for compatibility with gfortran compilers
- Fixed bugs for compatibility with intel compilers
- Fixed bug in standalone model for computing ice water path input to Cloud_JX

### Removed
- Removed unnecessary whitespace printed to log
- Removed CLDFLAG, LNRG, and NRANDO from CLOUD_JX argument list since they are global variables
- Removed unused arguments in ICA_* subroutines
- Removed unused common variables JXL_, JXL1_, and JXL2_
- Removed redundant file close in initialization

## [7.7] - committed 2021-09-01
### Changed
- Replace files with version 7.7 from M. Prather's ftp site

## [7.4d] - committed 2021-09-01
### Changed
- Replaced files with version 7.4d from M. Prather's ftp site

## [7.3e] - committed 2021-09-01
### Added
- Added files for version 7.3e from M. Prather's ftp site
15 changes: 13 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,16 @@ endif()
#-----------------------------------------------------------------------------
if(CLOUDJ_EXTERNAL_CONFIG AND GCCLASSIC_WRAPPER)
target_compile_definitions(CloudJBuildProperties
INTERFACE MODEL_GCCLASSIC
INTERFACE MODEL_GEOSCHEM
)
endif()

#-----------------------------------------------------------------------------
# For Cloud-J as submodule in GEOS-Chem "High Performance" (GCHP) superproject
#-----------------------------------------------------------------------------
if(CLOUDJ_EXTERNAL_CONFIG AND GCHP_WRAPPER)
target_compile_definitions(CloudJBuildProperties
INTERFACE MODEL_GEOSCHEM
)
endif()

Expand All @@ -271,12 +280,14 @@ endif()
if(CLOUDJ_EXTERNAL_CONFIG AND MAPL_ESMF)
target_link_libraries(CloudJBuildProperties
INTERFACE $<LINK_ONLY:MAPL.base>
INTERFACE $<LINK_ONLY:MAPL.generic>
)
target_include_directories(CloudJBuildProperties
INTERFACE $<TARGET_PROPERTY:MAPL.base,INTERFACE_INCLUDE_DIRECTORIES>
INTERFACE $<TARGET_PROPERTY:MAPL.generic,INTERFACE_INCLUDE_DIRECTORIES>
)
target_compile_definitions(CloudJBuildProperties
INTERFACE ESMF_
INTERFACE MAPL_ESMF
)
endif()

Expand Down
12 changes: 6 additions & 6 deletions src/Core/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
# Cloud-J/src/Core/CMakeLists.txt

add_library(CloudJ_Core STATIC EXCLUDE_FROM_ALL
cldj_cmn_mod.f90
cldj_error_mod.f90
cldj_init_mod.f90
cld_sub_mod.f90
fjx_sub_mod.f90
osa_sub_mod.f90
cldj_cmn_mod.F90
cldj_error_mod.F90
cldj_init_mod.F90
cldj_sub_mod.F90
cldj_fjx_sub_mod.F90
cldj_osa_sub_mod.F90
)
target_link_libraries(CloudJ_Core
PUBLIC CloudJBuildProperties
Expand Down
Loading