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

String padding #1621

Closed
axelboc opened this issue Apr 22, 2024 Discussed in #1612 · 1 comment · Fixed by #1659
Closed

String padding #1621

axelboc opened this issue Apr 22, 2024 Discussed in #1612 · 1 comment · Fixed by #1659
Labels
enhancement New feature or request

Comments

@axelboc
Copy link
Contributor

axelboc commented Apr 22, 2024

Discussed in #1612

Originally posted by benajamin April 15, 2024
One of the difficulties new adopters of HDF5 come across is that there are many ways to write strings (fixed vs variable length, null terminated or not). Many tools don't clearly tell what they are reading or writing, and some programs will crash or malfunction if they don't get an expected string type. The hdfview utility is the one that I find most useful for debugging string issues and it would be great if the h5web viewer could also provide this detailed string info in the "inspect" mode and therefore help debug HDF5 string bugs.

Originally posted by benajamin April 20, 2024
The string padding info would be great to have but can be hidden behind the Inspect accordion since it is probably only interesting for debugging. I don't think that there is a definite right or wrong type of padding to use, I have just seen incompatibilities between programs reading and writing HDF5 that caused bugs that were difficult to diagnose because the critical padding information was not easy to expose.

@axelboc axelboc added the enhancement New feature or request label Apr 22, 2024
@axelboc
Copy link
Contributor Author

axelboc commented Apr 22, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant