Skip to content

cmake cleanup in CCPP; cmake in FV3dycore#385

Closed
aerorahul wants to merge 13 commits into
NOAA-EMC:developfrom
aerorahul:feature/cmake_only_in_dycore_of_emc
Closed

cmake cleanup in CCPP; cmake in FV3dycore#385
aerorahul wants to merge 13 commits into
NOAA-EMC:developfrom
aerorahul:feature/cmake_only_in_dycore_of_emc

Conversation

@aerorahul
Copy link
Copy Markdown
Contributor

@aerorahul aerorahul commented Sep 10, 2021

Description

This PR:

Testing

How were these changes tested? RT's
What compilers / HPCs was it tested with? RT's
Are the changes covered by regression tests? Yes.
Have the ufs-weather-model regression test been run? On what platform? In progress

  • Will the code updates change regression test baseline? No
  • Please commit the regression test log files in your ufs-weather-model branch

Dependencies

If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)

Do PRs in upstream repositories need to be merged first?
If so add the "waiting for other repos" label and list the upstream PRs

Comment thread ccpp/CMakeLists.txt Outdated
@junwang-noaa
Copy link
Copy Markdown
Collaborator

Do we have CCPP PRs for this fv3 PR? can we list them in the Dependencies section?

@aerorahul
Copy link
Copy Markdown
Contributor Author

Do we have CCPP PRs for this fv3 PR? can we list them in the Dependencies section?

As soon as @climbfuji opens the relevant PR's, I will update the description here.

@climbfuji
Copy link
Copy Markdown
Collaborator

Do we have CCPP PRs for this fv3 PR? can we list them in the Dependencies section?

As soon as @climbfuji opens the relevant PR's, I will update the description here.

These PRs were opened many weeks ago ...

Associated PRs (see also ufs-weather-model PR):

NOAA-GFDL/GFDL_atmos_cubed_sphere#141
NCAR/ccpp-framework#382
NCAR/ccpp-physics#705
#385
ufs-community/ufs-weather-model#791

@aerorahul
Copy link
Copy Markdown
Contributor Author

Do we have CCPP PRs for this fv3 PR? can we list them in the Dependencies section?

done.

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.

This PR, together with NOAA-GFDL/GFDL_atmos_cubed_sphere#141, changed the regression test results on AWS (CI) and Jet with Intel for yet unknown reasons.

@aerorahul
Copy link
Copy Markdown
Contributor Author

The atmosphere dycore build is treated differently than the other components of the UFS.
Isolating and fixing this issue will likely result in non-reproducible baseline due to this special handling.

@aerorahul aerorahul closed this Sep 14, 2021
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