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

netcdf write optimizations #19

Closed
wants to merge 4 commits into from
Closed

Conversation

jswhit2
Copy link
Collaborator

@jswhit2 jswhit2 commented Dec 5, 2019

see #18

Included in #41

ncerr = nf90_def_var(ncid, trim(fldName), NF90_FLOAT, &
(/im_dimid,jm_dimid,pfull_dimid,time_dimid/), varids(i), &
shuffle=.false.,deflate_level=ideflate,&
chunksizes=(/im,jm,1,1/),cache_size=40*im*jm); NC_ERR_STOP(ncerr)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Must the chunksizes be /im,jm,1,1/?

Having chunksizes of 1 performs badly. Having chunksizes that are the entire extent of the dimension can be fast for smaller sizes, but larger sizes should be broken up.

Copy link
Contributor

Choose a reason for hiding this comment

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

No, I just set it to that because it seemed like the most common access pattern (reading a 2-d horizontal grid).

@jswhit2 jswhit2 mentioned this pull request Jan 10, 2020
@junwang-noaa
Copy link
Collaborator

Code commit in #48

climbfuji pushed a commit to climbfuji/fv3atm that referenced this pull request Apr 24, 2020
…_mods

Bug fixes to 3D diagnostic tendencies (based on NOAA-EMC#15)
junwang-noaa pushed a commit to junwang-noaa/fv3atm that referenced this pull request Sep 30, 2021
LarissaReames-NOAA pushed a commit to LarissaReames-NOAA/fv3atm that referenced this pull request Nov 17, 2023
* Fix for multi_gases to 32 bit compiling
* Add a subroutine to read multi_gases_nml to be consistent with others
* Replace rilist and cpilist with ri and cpilist for multi_gases_nml
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