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

Pass CollectionType when reading #244

Open
Flamefire opened this issue Jun 23, 2016 · 1 comment
Open

Pass CollectionType when reading #244

Flamefire opened this issue Jun 23, 2016 · 1 comment

Comments

@Flamefire
Copy link
Contributor

HDF5 does datatype conversion when required (https://www.hdfgroup.org/HDF5/doc/UG/HDF5_Users_Guide-Responsive%20HTML5/index.html#t=HDF5_Users_Guide%2FDatatypes%2FHDF5_Datatypes.htm%23TOC_6_10_Data_Transferbc-26&rhtocid=6.5_2)
So the interfaces should accept a CollectionType when reading making the read/write signature almost the same.

Example:

    virtual void readAttribute(int32_t id,
            const CollectionType& type,
            const char *dataName,
            const char *attrName,
            void *buf,
            Dimensions *mpiPosition = NULL) = 0;
    virtual void writeAttribute(int32_t id,
            const CollectionType& type,
            const char *dataName,
            const char *attrName,
            const void *buf) = 0;

If one is really interested in the actual collection type, there is the readAttributeMeta function. One could also make the type a pointer, with NULL allowed which would be handy for strings.

This was referenced Jun 23, 2016
@ax3l ax3l added the feature label Jul 8, 2016
@ax3l ax3l added this to the Version 1.5/1.4.X milestone Jul 8, 2016
@ax3l ax3l added the API change label Jul 8, 2016
@ax3l
Copy link
Member

ax3l commented Jul 8, 2016

one general comment: a current design flaw is still, that native HDF5 types such as hid_t, hsize_t, ... are currently exposed in the public API of libSplash (see e.g., CollectionType, Dimensions, Serial&ParallelDataCollector, the new DCAttributeInfo). This leads to the problem, that booth <splash/splash.h> and <hdf5.h> need to be present in user code.

Ideally, <hdf5.h> and objects / types from it would not be visible in any of the user-included header files of libSplash.

@ax3l ax3l changed the title Pass collectiontype when reading Pass CollectionType when reading Jul 8, 2016
@ax3l ax3l modified the milestones: Future, Version 1.5/1.4.X Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants