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

Cleanup constants in all physics #104

Closed
climbfuji opened this issue Jun 26, 2018 · 7 comments
Closed

Cleanup constants in all physics #104

climbfuji opened this issue Jun 26, 2018 · 7 comments

Comments

@climbfuji
Copy link
Collaborator

samfdeepcnv_run should get its various physical constants through the argument list as per CCPP requirements.

@grantfirl
Copy link
Collaborator

also samfshalcnv_run

@climbfuji climbfuji changed the title New scheme samfdeepcnv uses constants from physcons.f90 New schemes samfdeepcnv and gfdlmp use constants from physcons.F90 Aug 14, 2018
@climbfuji climbfuji changed the title New schemes samfdeepcnv and gfdlmp use constants from physcons.F90 New schemes samfdeepcnv, samfshalcnv_run and gfdlmp_{init,run} use constants from physcons.F90 Aug 14, 2018
@climbfuji
Copy link
Collaborator Author

also gfdlmp_{init,run}

@climbfuji
Copy link
Collaborator Author

this will also be true for the CAPS New Tiedtke physics scheme cu_ntiedtke.F90 (see PR #201)

@climbfuji climbfuji changed the title New schemes samfdeepcnv, samfshalcnv_run and gfdlmp_{init,run} use constants from physcons.F90 Cleanup constants in all physics Nov 18, 2019
@climbfuji
Copy link
Collaborator Author

This applies to many other physics schemes, someone should go through all of them and clean up. Push some common ones (rainmin, ...) to physcons.F90 and pass through argument list.

@climbfuji
Copy link
Collaborator Author

See also #245

@climbfuji
Copy link
Collaborator Author

See also #304 and #318.

Also, subroutines called by satmedmfvdif(q) use imported constants (the CCPP entry points do not).

@climbfuji
Copy link
Collaborator Author

This catch-all issue for importing constants instead of passing them via the argument list doesn't help. There are many issues related to this problem on a per-scheme basis, which makes more sense.

hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this issue Aug 3, 2022
…owup bugfix (NCAR#104)

* point to ccpp branch chsp_latlon_post

* add chsp code changes from Jongil

Co-authored-by: Jongil Han <[email protected]>

* add landsfcmdl variable to model sfc history file meta data

* add global latlon grid on write grid component, code changes from Jeffrey Whitaker

Co-authored-by: Jeffrey Whitaker <[email protected]>

* add chsp code changes in ccpp-physics

Co-authored-by: Jongil Han <[email protected]>

* remove support for legacy Intel compilers in CCPP gmake build

Co-authored-by: Dom Heinzeller <[email protected]>

* update ccpp physics, fix syntax error

* update sflx.f in ccpp physics

* CCN-IN followup bugfix

Co-authored-by: Grant Firl <[email protected]>

* Bugfix for GNU compiler in io/module_wrt_grid_comp.F90; bugfix in gfsphysics/physics/sfc_diff.f; updates for CCPP

* merge fv3 dycore ccpp framework and physics to the latest repoes

* cmeps integration changes

Co-authored-by: Denise Worthen <[email protected]>

* update CCPP with tsfcl change

* fix timestr in module_cap_cpl.F90

* fix syntax error in ugwp_driver_v0.F

* point ccpp/physics back to NCAR master

Co-authored-by: Jun Wang <[email protected]>
Co-authored-by: Jeffrey Whitaker <[email protected]>
Co-authored-by: Jongil Han <[email protected]>
Co-authored-by: Dom Heinzeller <[email protected]>
Co-authored-by: Grant Firl <[email protected]>
Co-authored-by: Dom Heinzeller <[email protected]>
Co-authored-by: Denise Worthen <[email protected]>
hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this issue Aug 3, 2022
…erosols

Gf radiation update revert aerosols
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants