-
Notifications
You must be signed in to change notification settings - Fork 55
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
PIO 2.5.4 and NetCDF-4.8.1 (HDF5-1.12.1) #1889
Comments
@edwardhartnett Can you help with this issue? |
Try building PIO without the --enable-netcdf-integration and see if that works... |
@edwardhartnett that does work, the issue is supporting the NC_DISPATCH_VERSION=3 |
@edwardhartnett I would like to make a new PIO tag - any chance we can make NC_DISPATCH_VERSION=3 |
Sorry, I didn't have time and may not until the next release of netcdf-c, in which the dispatch table will change again. What I am doing is working on new compression in netCDF, and this will also require some new functions in PIO to use these capabilities... |
Resolved |
I have not tried auto-tools, but cmake (v3.19.1) configure fails with NetCDF-4.8.1 because
NC_DISPATCH_VERSION
has changed from 2 to 3 in the latest release. I changed line 369 inCMakeLists.txt
to:#if !(NC_DISPATCH_VERSION == 2 || NC_DISPATCH_VERSION == 3)
and configure completed and make, make tests completed but I get 7 failed tests with ctest:
I found this pull request and this issue about
NC_DISPATCH_VERSION
so I'm betting you already know some changes (possibly tosrc/ncint/ncintdispatch.{c&h}
) are probably necessary to account for the switch fromfilter_actions
tofilter_ids
andfilter_info
. Unfortunately, I don't know nearly enough about filters or dispatch tables to help much.The text was updated successfully, but these errors were encountered: