Skip to content
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

Update chunksizes for cubed sphere grid #876

Merged
merged 5 commits into from
Oct 23, 2024

Conversation

DusanJovic-NOAA
Copy link
Collaborator

Description

Change the default chunking for the cubed sphere history files to use only the horizontal grid size of each of the tiles.

What bug does it fix, or what feature does it add? Slow writing of native grid history files, especially with c768 configurations.
Is a change of answers expected from this PR? No.

This PR should be combined with other PR that does not change baselines.

Issue(s) addressed

Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues should always be created before starting work on a PR branch!)

Testing

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

  • Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
  • Please commit the regression test log files in your ufs-weather-model branch

Dependencies

No.

Requirements before merging

  • All new code in this PR is tested by at least one unit test
  • All new code in this PR includes Doxygen documentation
  • All new code in this PR does not add new compilation warnings (check CI output)

@jkbk2004
Copy link
Collaborator

@DusanJovic-NOAA can you sync up branch and combine in #878 to point to Dom's ccpp-framework branch?

@DusanJovic-NOAA
Copy link
Collaborator Author

@DusanJovic-NOAA can you sync up branch and combine in #878 to point to Dom's ccpp-framework branch?

Done. @climbfuji please double check.

@climbfuji
Copy link
Collaborator

@DusanJovic-NOAA can you sync up branch and combine in #878 to point to Dom's ccpp-framework branch?

Done. @climbfuji please double check.

Thanks, looks right!

@jkbk2004
Copy link
Collaborator

@DusanJovic-NOAA ccpp-framework pr was merged. New hash is NCAR/ccpp-framework@b7d55fd

@DusanJovic-NOAA
Copy link
Collaborator Author

@DusanJovic-NOAA ccpp-framework pr was merged. New hash is NCAR/ccpp-framework@b7d55fd

Done.

@jkbk2004 jkbk2004 merged commit 99091f9 into NOAA-EMC:develop Oct 23, 2024
2 of 6 checks passed
@DusanJovic-NOAA DusanJovic-NOAA deleted the cubed_sphere_chunks branch October 24, 2024 11:25
tsga added a commit to tsga/fv3atm that referenced this pull request Oct 30, 2024
* develop:
  Update chunksizes for cubed sphere grid (NOAA-EMC#876)
  First reconciliation PR from production/RRFS.v1 (NOAA-EMC#872)
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.

writing native grid atmf history files is too slow in FV3ATM
4 participants