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

UFS-dev PR#184 #1073

Merged
merged 53 commits into from
Jun 4, 2024
Merged

UFS-dev PR#184 #1073

merged 53 commits into from
Jun 4, 2024

Conversation

DusanJovic-NOAA and others added 30 commits March 24, 2022 14:45
Switch from 'use mpi' to 'use mpi_f08'
DusanJovic-NOAA and others added 20 commits January 16, 2024 19:31
…eta: change units 'flashes 5 min-1' to 'flashes min-1' and update long name to make clear this is per 5 minutes
…90, scale lightning threat from flashes per 5 minutes to flashes per minute to match units in metadata
correctly convert from flashes per five minutes to flashes per minute
incorporate csawMG update and pass both intel and GNU RT tests
Resolve various subroutine argument mismatches
@mkavulich
Copy link
Collaborator

@grantfirl Does this update require SCM changes as well? When I updated my SCM branch to use this physics branch, prebuild fails with an error:

DEBUG: Requested variable zenith_angle_temporal_adjustment_factor_for_shortwave_fluxes in MODULE_ysuvdif SCHEME_ysuvdif SUBROUTINE_ysuvdif_run matched to target physics%Interstitial%xmu in module CCPP_typedefs
Traceback (most recent call last):
  File "/Volumes/d1/kavulich/CCPP/rename_suite_files/grants_pr/mybranch/ccpp-scm/ccpp/framework/scripts/ccpp_prebuild.py", line 827, in <module>
    main()
  File "/Volumes/d1/kavulich/CCPP/rename_suite_files/grants_pr/mybranch/ccpp-scm/ccpp/framework/scripts/ccpp_prebuild.py", line 783, in main
    raise Exception('Call to compare_metadata failed.')
Exception: Call to compare_metadata failed.

@mkavulich
Copy link
Collaborator

Sorry, I just now saw your comment here, disregard

@grantfirl grantfirl merged commit 76cf93f into NCAR:main Jun 4, 2024
3 checks passed
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

Successfully merging this pull request may close these issues.

8 participants