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

directory -> group/path #1

Open
nilsbecker opened this issue Sep 29, 2015 · 5 comments
Open

directory -> group/path #1

nilsbecker opened this issue Sep 29, 2015 · 5 comments

Comments

@nilsbecker
Copy link

great to see that hdf5 support is finally coming to ocaml!

on cursory reading of the highlevel api and docs i often see the term 'directory'. while this is of course intuitive, the standard terminology is path if it's the string or 'group' (and/or 'dataset') if it's the object i believe.

https://www.hdfgroup.org/HDF5/doc/Glossary.html#Glossary-Group

i know hdf5 via h5py which is a joy to use. this is of course based on numpy and heavily uses overloading of slicing syntax, which is probably not a good match for ocaml. but there might be other aspects where it could serve as inspiration for the api.

https://github.com/h5py/h5py/tree/master/h5py/_hl

some ideas for the highlevel interface (may be nonsense and/or already implemented, sorry if so.)

  • i would have expected a single file creation function with some variant argument for the file mode. but i guess that's taste?
  • would it be good to provide a 'with file' construct/function which would work like a context manager in python?
  • fold/iter/map functions over hdf5 groups? (this may be nonsense since the links in an hdf5 file can be duplicate and circular as i just found out)
  • hdf5 capabilities depend on the format version a lot. highlevel verification of version requirements for opened files?

disclaimer: have not actually used, only looked at the .mli's

@nilsbecker nilsbecker changed the title directory -> group directory -> group/path Sep 29, 2015
@vbrankov
Copy link
Owner

Thanks for the feedback. I've replaced "directory" with "group".

@struktured
Copy link
Contributor

@vbrankov I still see the term directory in the high level api.

@vbrankov
Copy link
Owner

I grepped the code for "irectory" and cannot find it. Where do you see it?

@struktured
Copy link
Contributor

struktured commented Oct 14, 2017

@vbrankov
Copy link
Owner

Fixed.

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

No branches or pull requests

3 participants