Skip to content

Conversation

@domingom
Copy link
Collaborator

@domingom domingom commented Apr 11, 2025

Expected Differences

  • Do these changes introduce new tools, command line arguments, or configuration file options? [No]

    If yes, please describe:

Pull Request Testing

  • Describe testing already performed for these changes:

    Numerous tests including previous Examples from tutorials(1,2,3,7) and a new tutorial example (8) to be introduced in another PR.

  • Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

    None

  • Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]
    Documentation will be introduced in a subsequent PR.

  • Do these changes include sufficient testing updates? [Yes or No]
    N/A

  • Will this PR result in changes to the test suite? [Yes or No]

    If yes, describe the new output and/or changes to the existing output:

    N/A

  • Please complete this pull request review by [Fill in date].

    4-11-2025

Pull Request Checklist

  • Complete the PR definition above.
  • Ensure the PR title matches the feature or bugfix branch name.
  • Select: Reviewer(s)
  • After submitting the PR, select the ⚙️ icon in the Development section of the right hand sidebar. Search for the issue that this PR will close and select it, if it is not already selected.
  • After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
  • Close the linked issue and delete your feature or bugfix branch from GitHub.

jprestop and others added 30 commits August 9, 2024 16:22
First set of CP method code additions (does not include TVC).
python coupling tools (1/3): GeoSpec
…LCD land cover to z0m clook up table and geospec.json file
…tric tensor terms to accomodate terrain following coordinate frame.
… the formulation of cell face velocties (for advection terms) and calculation of pressure gradient force components.
…aing-following coordinate in sgsTKE prognostic equation and sgs Turbulence closure terms.
Bugfix to get the terrain following coordinate actually in place
… adding self-consistent formulations of th0 (omitting any use of pres_grnd) in surface layer schemes.
feature/fe coupling Initial phase
…ut files, also ensured strict update of master simTime to be a perfect multiple of dt (avoiding roundoff error in simTime) and implemented the ability to read/write int variables in the netcdf and binary modes of the IO module to allow for upcoming TVCP parameters.
…files

feature/time_variable_in_output_files
…bmodule and added a few small code cleanup modifications.
jsauer-NCAR and others added 13 commits April 1, 2025 17:15
…h new formulation for u,v, and w that lack a base state description and to intrdoduce the ceilingAdvectionBC parameter to accomodate coupling and turn off rigid-lid that sets w=0 at the ceiling.
…ments

feature/final_coupling_BCs_adjustments
…MPI_Barrier following final output file write to ensure rank-synchronized cleanup and exit on simulation end, and provide missing broadcast of scalar float variables when reading a NetCDF restart file.
…g/gathering various fields during NetCDF IOis reset to zero everywhere before use in the scatter gather of each registered variable field. This prevents inadvertent pollution of halo cells observed in some fields (e.g. hydroauzscalars at the western boudnary on restart).
…tained them in the CUDA submodules of HYDRO_CORE.
Wrf coupling python -> final adjustments to the pre-processing utilities
…nd., and true use of iol in exchange coeffs)
Copy link
Collaborator

@jsauer-NCAR jsauer-NCAR left a comment

Choose a reason for hiding this comment

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

All commits in this PR were individually reviewed in the originating forked repository branch.

@jsauer-NCAR jsauer-NCAR merged commit 8d204c4 into NCAR:develop Apr 11, 2025
1 check passed
@domingom domingom mentioned this pull request Apr 11, 2025
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants