GEOPY-2756: H5 flags to optimize performances

[domfournier]

GEOPY-2756 - H5 flags to optimize performances

[Copilot]

Pull request overview

This PR updates how new .geoh5 files are created to use specific HDF5 library version bounds (libver) as part of a performance-optimization effort, and adds a regression test to validate the new creation settings.

Changes:

• Set libver=("v110", "v114") when creating new on-disk workspaces via h5py.File(...).
• Update test_empty_workspace to use a context manager and assert the created file’s libver bounds.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
geoh5py/workspace/workspace.py	Adds explicit HDF5 libver bounds when creating new on-disk .geoh5 files.
tests/workspace_test.py	Adjusts workspace creation test and asserts the new libver behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.24%. Comparing base (0f3a0d9) to head (51db7ea).

Additional details and impacted files

@@               Coverage Diff               @@
##           release/GA_4.8     #900   +/-   ##
===============================================
  Coverage           91.23%   91.24%           
===============================================
  Files                 113      113           
  Lines               10508    10507    -1     
  Branches             1944     1943    -1     
===============================================
  Hits                 9587     9587           
  Misses                486      486           
+ Partials              435      434    -1     

Files with missing lines	Coverage Δ
geoh5py/workspace/workspace.py	91.45% <100.00%> (+0.14%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sebhmg]

see feedback from copilot: same options must be passed when creating as BytesIO

[domfournier]

See response

[sebhmg]

see more parameters to pass when creating h5 on BytesIO

[sebhmg]

would need the other options too.

Suggested change

	self._geoh5 = h5py.File(self.h5file, "a", libver=("v110", "v114"))
	self._geoh5 = h5py.File(
	self.h5file,
	"a",
	fs_strategy="page",
	page_buf_size=DEFAULT_PAGE_BUF_SIZE,
	libver=("v110", "v114"),
	)

or just merge both case together:

Suggested change

	self._geoh5 = h5py.File(self.h5file, "a", libver=("v110", "v114"))
	if isinstance(self.h5file, (BytesIO, Path)):
	self._geoh5 = h5py.File(
	self.h5file,
	"a" if isinstance(self.h5file, BytesIO) else "x",
	fs_strategy="page",
	page_buf_size=DEFAULT_PAGE_BUF_SIZE,
	libver=("v110", "v114"),
	)

self._geoh5 = h5py.File(self.h5file, "a", libver=("v110", "v114"))