You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The main HDF5.jl file is rather large, and very difficult to navigate, especially since the API is not yet fully documented. I think this would be easier to develop and maintain if we were to split this up into smaller files. I originally tried this in #833, but it was quite a big change, so instead I decided to tackle it smaller chunks.
The question is what should go where. @mkitti suggested I write it up as an issue so that it can be discussed in one place.
My overall idea is to split it out according to the how the C API is broken up
The main HDF5.jl file is rather large, and very difficult to navigate, especially since the API is not yet fully documented. I think this would be easier to develop and maintain if we were to split this up into smaller files. I originally tried this in #833, but it was quite a big change, so instead I decided to tackle it smaller chunks.
The question is what should go where. @mkitti suggested I write it up as an issue so that it can be discussed in one place.
My overall idea is to split it out according to the how the C API is broken up
h5p
): done in Refactor Properties interface #847h5f
):File
type andh5open
: move file-related functions to new file #970h5g
):Group
type andsetindex!
/getindex
overloads: move group and link functions to new file #971h5t
):Datatype
and conversions between Julia types and HDF5 datatypes (move datatype functions to new file #975)h5s
):Dataspace
and conversions between Julia indexing and dataspaces: See move dataspace-related functions to new file #961h5d
):Dataset
andread
/write
functionality: see Move code into datasets.jl, readwrite.jl, and types.jl #955.h5a
): See move attributes to new file, add docs #943h5r
):Reference
type and dereferencing: see move Reference pieces to new file #968h5o
): See move object-related functions to new file #976The text was updated successfully, but these errors were encountered: