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

Testings imple szip filter setup Unexpected Result #2420

Open
WardF opened this issue Jun 22, 2022 · 8 comments
Open

Testings imple szip filter setup Unexpected Result #2420

WardF opened this issue Jun 22, 2022 · 8 comments
Assignees
Milestone

Comments

@WardF
Copy link
Member

WardF commented Jun 22, 2022

Running on MacOS, with HDF5 built w/out szlib support, is seeing a failure in nc_test4/tst_vars3.c, line: 443. The message being reported is:

**** testing simple szip filter setup...Sorry! Unexpected result (...)

This occurs even if ENABLE_FILTER_TESTING=OFF is specified when running cmake. I'm uncertain what the solution is, but it seems that we should be able to selectively toggle which filter libraries to use, even when detected, perhaps?

@WardF WardF added this to the 4.9.1 milestone Jun 22, 2022
@WardF WardF self-assigned this Jun 22, 2022
@WardF
Copy link
Member Author

WardF commented Jun 22, 2022

Note: This only occurs using cmake.

@edwardhartnett
Copy link
Contributor

Make sure that HAVE_H5Z_SZIP is undefined if szip is not present in HDF5.

@WardF
Copy link
Member Author

WardF commented Jun 22, 2022

I'm wondering if this block of nc_test4/tsts_vars3.c should be split off into a new test file that can be fenceposted via ENABLE_FILTER_TESTING; it feels like szip should not be part of the equation in this case. Or, perhaps change the current #ifdef HAVE_SZ to #ifdef HAVE_H5Z_SZIP; that feels more apt, for this file, but as always I'll benefit from considering it a few moments before asserting an answer.

@WardF
Copy link
Member Author

WardF commented Jun 22, 2022

Rather, as always, I'll wish I had considered it before asserting an answer, if I don't.

@edwardhartnett
Copy link
Contributor

Well this test is a holdover from the case of HDF5 having szip, but before filters were supported.

In other words this test can run when HDF5 supports szip, even if filter testing is turned off.

I think it would also be fine to just not test this if filter testing is off, if that is easier.

@DennisHeimbigner
Copy link
Collaborator

Ward- can you send me the contents of libnetcdf.settings when
you see the MacOS fail?

@WardF
Copy link
Member Author

WardF commented Jun 22, 2022

@DennisHeimbigner sure thing:

DennisHeimbigner added a commit to DennisHeimbigner/netcdf-c that referenced this issue Jun 22, 2022
re: Unidata#2420

nc_test4/tst_vars3.c has the wrong conditional
around the szip tests (did I do that?).
Anyway, the current test is
> #ifdef HAVE_SZ

and it should be
> #ifdef HAVE_H5Z_SZIP

because the only thing that matters is that HDF5 lib has szip support.
@DennisHeimbigner
Copy link
Collaborator

May be fixed by PR #2421

@WardF WardF modified the milestones: 4.9.1, 4.9.2 Feb 13, 2023
@WardF WardF modified the milestones: 4.9.2, 4.9.3 May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants