Skip to content

Bugfixes from public release#112

Merged
DusanJovic-NOAA merged 9 commits into
NOAA-EMC:developfrom
DusanJovic-NOAA:bugfixes_from_public_release
May 8, 2020
Merged

Bugfixes from public release#112
DusanJovic-NOAA merged 9 commits into
NOAA-EMC:developfrom
DusanJovic-NOAA:bugfixes_from_public_release

Conversation

@DusanJovic-NOAA
Copy link
Copy Markdown
Collaborator

@DusanJovic-NOAA DusanJovic-NOAA commented May 7, 2020

Cherry pick some bug fixes and cleanup commits from release/public-v1 branch.

Associated PRs:
ufs-community/ufs-weather-model#119

jedwards4b and others added 9 commits May 5, 2020 13:20
make variables using filename_base consistant in length
* set default calendar to gregorian
* set default values of write_nemsioflip and write_fsyncflag to .true.
Bring hotfixes from release/public-v1 back to master (CCPP changes)
…ease-dom"

This reverts commit bd142d4, reversing
changes made to d5fda8a.
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Maybe I am doing something wrong, but I don't see the changes from my commits when I click on "all commits"? There should be several in ccpp/, one on GFS_physics_driver.F90, one in .gitmodules?

@climbfuji
Copy link
Copy Markdown
Collaborator

Maybe I am doing something wrong, but I don't see the changes from my commits when I click on "all commits"? There should be several in ccpp/, one on GFS_physics_driver.F90, one in .gitmodules?

Ah, now I see, this is what you had to revert. I'll go and have a good look at the CCPP changes then.

Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Approved, but I do not understand why/how the regression tests failed when you merged my PR.

@DusanJovic-NOAA DusanJovic-NOAA merged commit 63658ed into NOAA-EMC:develop May 8, 2020
@DusanJovic-NOAA DusanJovic-NOAA deleted the bugfixes_from_public_release branch May 8, 2020 14:11
climbfuji pushed a commit to climbfuji/fv3atm that referenced this pull request May 8, 2020
climbfuji pushed a commit to climbfuji/fv3atm that referenced this pull request Oct 21, 2021
SamuelTrahanNOAA referenced this pull request in SamuelTrahanNOAA/fv3atm Jun 13, 2022
* Add the option to run regression test with MULTI_GASES=Y
* Add multi_gases CCPP regression test
LarissaReames pushed a commit to LarissaReames/fv3atm that referenced this pull request Nov 17, 2023
Documentation and defaults changes
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.

4 participants