Sync develop changes August 21 - 27 to hdf5_1_14 #4774
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Use gfortran 14 for cmake-test on mac * Remove notarization step * Address @byrnHDF review
Removes a duplicated HDopen macro from the performance testing programs
* Add minimal amount of S3 request logging to ROS3 * Fix ROS3 logging ifdef conditions
This PR is part of the incremental switching H5I_object() and H5I_object_verify() to their VOL counterpart, H5VL_object() and H5VL_vol_object_verify(), a newly addedinternal function. Fixes HDFGroupGH-4730 partially.
- Verified that the listed functions do not include null terminator in the returned length - Improved some of the tests - Corrected documentation Fixes HDFGroupGH-4704 * Casted a positive int to size_t
The code can't be downloaded due to changes that put it behind an EarthData login. We'll disable this while we figure out a work-around.
This PR switches H5I_object_verify() to H5VL_vol_object_verify() in the H5F API and fixes documentation of H5Fmount and H5Funmount. * More on H5F API
The t_pmulti_dset and t_select_io_dset tests rely on the behavior of the previous private rand_r-like implementation to get the correct sequence of random numbers to pass. This has been restored using a fully private rand_r-like implementation that doesn't rely on rand_r and will work on Windows and other platforms where rand_r doesn't exist.
Don't run parallel tests in both Autotools and CMake with multiple processes. ph5diff still runs with -j2 w/ Autotools since the test script is in the tools/test/h5diff directory. * Split off AOCC CMake parallel tests * Remove unnecessary NPROCS env vars * Put NPROCS back in serial tests We run ph5diff tests there
This PR switches H5I_object_verify() to H5VL_vol_object_verify() in the H5G API and removes unnecessary casts.
* Add Fortran and C++ to Autotools * Add Fortran and C++ to Linux CMake * Add C++ to Windows CMake * Fix bad GitHub workspace variable
* Don't skip file tests * Remove test with invalid flag for H5Fopen * Verify that create/open of unseekable file fails * Remove failure verification
Turns on testing, both serial and parallel, but skips: * dt_arith and dtransform in CMake * All main library tests in the Autotools Due to dt_arith and dtransform segfaults when handling long doubles.
'bit endian' --> 'big endian'
lrknox
requested review from
derobins,
byrnHDF,
fortnern,
jhendersonHDF,
qkoziol,
vchoi-hdfgroup,
bmribler,
glennsong09 and
mattjala
as code owners
byrnHDF
reviewed
Aug 27, 2024
.github/workflows/julia-auto.yml
Outdated
| @@ -0,0 +1,79 @@ | |||
| name: hdf5 dev autotools julia | |||
byrnHDF
reviewed
Aug 27, 2024
.github/workflows/julia-cmake.yml
Outdated
| @@ -0,0 +1,82 @@ | |||
| name: hdf5 dev CMake julia | |||
byrnHDF
approved these changes
Aug 27, 2024
byrnHDF
added
Priority - 1. High 🔼
derobins
approved these changes
Aug 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.