-
-
Notifications
You must be signed in to change notification settings - Fork 265
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
Sync with develop #3764
Merged
Merged
Sync with develop #3764
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
Cleans up new files in Autotools `make distclean` in the test directory
This was only added to CMake many years ago and tests the tools library.
Cleans up h5dump and h5diff *.onion files in the Autotools when runing `make clean`.
Removes generated HDF5 and text output files when running `make clean`.
The flushrefresh_test directory was not being cleaned up w/ `make clean` under the Autotools
Some tests in tfile.c use h5_fileaccess to get a VFD-dependent file name but use the scheme from testhdf5, reusing the FILE1 and FILE8 names. This leads to files like test1.h5.h5 which are unintended and not cleaned up. This changes the filename scheme for a few tests to work with h5test, resulting in more informative names and allowing the files to be cleaned up at the end of the test. The test files have also been added to the `make clean` target for the Autotools.
Adds missing files to `make clean` for parallel, including Fortran.
* Add native VOL checks to deprecated functions * Remove unneeded native VOL checks * Move native checks to top level calls
* update stat arg for apple * use H5_HAVE_DARWIN for Apple ifdef * fix typo * removed duplicate H5_ih_info_t * added fortran async test to cmake
Adds a small cache of the first N bytes of a file opened with the read-only S3 (ros3) VFD, where N is 4kiB or the size of the file, whichever is smaller. This avoids a lot of small I/O operations on file open. Addresses GitHub issue HDFGroup#3381
* Update Autotools to correctly configure oneAPI Splits the Intel config files under the Autotools into 'classic' Intel and oneAPI versions, fixing 'unsupported option' messages. Also turns off `-check uninit` (new in 2023) in Fortran, which kills the H5_buildiface program due to false positives. * Enable Fortran in oneAPI CI workflow * Turn on Fortran in CMake, update LD_LIBRARY_PATH * Go back to disabling Fortran w/ Intel For some reason there's a linking problem w/ Fortran error while loading shared libraries: libifport.so.5: cannot open shared object file: No such file or directory
* added h5pget_actual_selection_io_mode_f test * added tests for h5pget_actual_selection_io_mode_f * fixed int_f type conversion
* add missing H5PGET_ACTUAL_SELECTION_IO_MODE_F dll
…DFGroup#3716) Add functions/callbacks for explicit control over chunk index open/close Add functions/callbacks to check if chunk index is open or not so that it can be opened if necessary before temporarily disabling collective metadata reads in the library Add functions/callbacks for requesting loading of additional chunk index metadata beyond the chunk index itself
…DFGroup#3761) Allow H5Pset_evict_on_close to be called regardless of whether a parallel build of HDF5 is being used Fail during file opens if H5Pset_evict_on_close has been set to true on the given File Access Property List and the size of the MPI communicator being used is greater than 1
jhendersonHDF
added
Merge
Use this label when a PR is for a downstream merge
Priority - 0. Blocker ⛔
This MUST be merged for the release to happen
Component - C Library
Core C library issues (usually in the src directory)
Component - Fortran
Fortran wrappers
Component - Parallel
Parallel HDF5 (NOT thread-safety)
Component - Testing
Code in test or testpar directories, GitHub workflows
Component - Build
CMake, Autotools
Type - Task
Actions that don't fit into any other type category
Branch - 1.14
labels
Oct 24, 2023
jhendersonHDF
requested review from
byrnHDF,
derobins,
brtnfld,
epourmal,
lrknox,
fortnern,
qkoziol,
vchoi-hdfgroup,
bmribler,
glennsong09 and
mattjala
as code owners
October 24, 2023 19:25
jhendersonHDF
commented
Oct 24, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change already made it into the branch, but that was merged after I picked all the commits from develop into this branch
byrnHDF
approved these changes
Oct 24, 2023
lrknox
approved these changes
Oct 24, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Component - Build
CMake, Autotools
Component - C Library
Core C library issues (usually in the src directory)
Component - Fortran
Fortran wrappers
Component - Parallel
Parallel HDF5 (NOT thread-safety)
Component - Testing
Code in test or testpar directories, GitHub workflows
Merge
Use this label when a PR is for a downstream merge
Priority - 0. Blocker ⛔
This MUST be merged for the release to happen
Type - Task
Actions that don't fit into any other type category
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.