Skip to content

Add tester for h5py interop via uv#26

Closed
vrv-bit wants to merge 3 commits intoscigolib:mainfrom
vrv-bit:main
Closed

Add tester for h5py interop via uv#26
vrv-bit wants to merge 3 commits intoscigolib:mainfrom
vrv-bit:main

Conversation

@vrv-bit
Copy link

@vrv-bit vrv-bit commented Mar 4, 2026

Adds a tester to run UV script to verify h5py can read the HDF files.

@kolkov
Copy link
Contributor

kolkov commented Mar 14, 2026

Thank you for putting this together, and especially for reporting #28 and #29 — your bug reports led to finding and fixing 3 real interoperability bugs, all shipped in v0.13.11.

That said, we're going to close this PR. Our testing strategy validates interoperability exclusively against the official HDF5 C tools (h5dump, h5ls, h5debug) from the HDF Group. h5py, while popular, is itself a third-party implementation wrapping the C library — so testing against it would be testing against someone else's interpretation of the format rather than the authoritative reference. Going forward, the official C toolchain is our single source of truth for format compliance.

We've added Go-native regression tests in v0.13.11 that cover this exact scenario (group + attribute + multiple datasets in non-alphabetical order) with round-trip verification.

Contributions are always welcome — feel free to open issues or PRs anytime!

@kolkov kolkov closed this Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants