Skip to content

Cleanup to ascii mesh converter #31

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

Conversation

matthewhoffman
Copy link
Member

@matthewhoffman matthewhoffman commented May 1, 2018

This merge fixes two minor issues in the Albany ASCII mesh converter, makes a couple minor changes to test case plotting scripts, and fixes some Registry typos.

These were added during writing of model description paper
* Fix ordering of velocity points
* add bed topo
@matthewhoffman matthewhoffman merged commit 5f9774e into MPAS-Dev:landice/develop May 1, 2018
@matthewhoffman matthewhoffman deleted the landice/update_mesh_converter branch May 1, 2018 18:09
mgduda pushed a commit to mgduda/MPAS-Model that referenced this pull request Aug 5, 2020
caozd999 pushed a commit to caozd999/MPAS-Model that referenced this pull request Jan 14, 2021
syha added a commit to syha/MPAS-Model that referenced this pull request Jul 11, 2022
Bug fixes in IAU merged into the latest version of release-stable

The major bug fix is to include the increment in dry density in updating the tendency for the coupled edge winds (ru). In the equation of d(r * u) = r * du + dr * u, the second term (dr*u) in RHS was missing in the original code. To add it back in, I also modified the code defining several new variables such as u, cellsOnEdge and rho_amb_edge.

mpas_atm_iau.F: A bug fixed in tend_ru (tendencies for coupled edge winds)
mpas_atm_core.F: A log message added for the IAU input stream 

Tests passed.
climbfuji pushed a commit to climbfuji/MPAS-Model that referenced this pull request Aug 25, 2022
Bug fixes in IAU merged into the latest version of release-stable

The major bug fix is to include the increment in dry density in updating the tendency for the coupled edge winds (ru). In the equation of d(r * u) = r * du + dr * u, the second term (dr*u) in RHS was missing in the original code. To add it back in, I also modified the code defining several new variables such as u, cellsOnEdge and rho_amb_edge.

mpas_atm_iau.F: A bug fixed in tend_ru (tendencies for coupled edge winds)
mpas_atm_core.F: A log message added for the IAU input stream 

Tests passed.
byoung-joo pushed a commit to byoung-joo/MPAS-Model that referenced this pull request Aug 26, 2022
Bug fixes in IAU merged into the latest version of release-stable

The major bug fix is to include the increment in dry density in updating the tendency for the coupled edge winds (ru). In the equation of d(r * u) = r * du + dr * u, the second term (dr*u) in RHS was missing in the original code. To add it back in, I also modified the code defining several new variables such as u, cellsOnEdge and rho_amb_edge.

mpas_atm_iau.F: A bug fixed in tend_ru (tendencies for coupled edge winds)
mpas_atm_core.F: A log message added for the IAU input stream

Tests passed.
LarissaReames pushed a commit to LarissaReames/MPAS-Model that referenced this pull request Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant