Skip to content

fix bug in UGWPv0 and v1#613

Merged
climbfuji merged 2 commits into
NCAR:masterfrom
grantfirl:ugwpv0_dependency_fix
Apr 28, 2021
Merged

fix bug in UGWPv0 and v1#613
climbfuji merged 2 commits into
NCAR:masterfrom
grantfirl:ugwpv0_dependency_fix

Conversation

@grantfirl
Copy link
Copy Markdown
Collaborator

@grantfirl grantfirl commented Apr 9, 2021

  • fix bug in UGWPv0 where it was using the wrong arguments in a oro_wam_2017() call
  • also cires_orowam2017.F90 appears to be a duplicate and is not used by any scheme
  • fix bug in UGWPv1 where it was using the wrong arguments in a oro_spectral_solver() call

…_2017() call; also cires_orowam2017.F90 appears to be a duplicate and is not used by any scheme
@grantfirl
Copy link
Copy Markdown
Collaborator Author

This has only been tested on Cheyenne/Intel with debug mode using the SCM build system, where the problem was discovered by @JulieSchramm .

@grantfirl grantfirl changed the title fix bug in UGWPv0 fix bug in UGWPv0 and v1 Apr 9, 2021
Copy link
Copy Markdown

@JulieSchramm JulieSchramm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch in the call to oro_spectral_solver!

climbfuji added a commit that referenced this pull request Apr 28, 2021
Collect PRs #613, #628, #634, #635 / update long names and vertical dimension for atmosphere_heat_diffusivity and atmosphere_momentum_diffusivity
@climbfuji climbfuji merged commit 54d8f07 into NCAR:master Apr 28, 2021
HelinWei-NOAA pushed a commit to HelinWei-NOAA/ccpp-physics that referenced this pull request Feb 26, 2023
…#613)

* Hardcoded coarse terrain for moving nest with terrain_smoother=4

* Update HAFSv1 CCPP suites to use the unified ugwp scheme.

* Update submodule upp.

* Removed debug print statements for terrain_smoother

* Fixed bug in calculation of nest halo weights identified by Biju Thomas in DDEBUG=ON builds.

* Updated fix to moving nest weight calculations.

* Use ind arrays for nest weight calculations. Ensures correct indexing.

* Clean up for the fix of using ind arrays for moving-nest weight calculations (from @wramstrom).

Co-authored-by: William Ramstrom <William.Ramstrom@noaa.gov>
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.

3 participants