This commit replaces all of the hdf5 C++ API calls with hdf5 C API. …

DeepLearnPhysics / larcv3

Third version of larcv. This is a complete replacement for larcv2.

MIT License

11 stars 6 forks source link

This commit replaces all of the hdf5 C++ API calls with hdf5 C API. … #37

Closed coreyjadams closed 5 years ago

coreyjadams commented 5 years ago

…This should allow larcv3 to get a little closer to proper MPI-IO behavior and optimizations. This is not verified yet, but the C++ API calls were a problem preventing this.

marcodeltutto commented 5 years ago

My last commit adds a new method to IOManager: int close_all_objects(hid_t fid); which closes all the open objects in a files, before closing a file. Otherwise, the Travis tests were failing. That is because in some tests we create a file and then we open it again in the same python session. Although the created file was closed, the objects in the file were still open, and so the file could not be opened again. Now all objets are closed before closing the file. The tests are passing.