-
Notifications
You must be signed in to change notification settings - Fork 163
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
Read datatype #796
Read datatype #796
Conversation
Thank you, this looks interesting and I'll have a closer look later. In the meantime:
is unfortunately a something that's a bit plagued by compiler errors, some compilers require this, while others insist that it's not present. Please see, for example. If CI complains, please adjust accordingly. |
Thank you for your quick reply. |
So the leak is not your responsibility (I think it was only not tested), and this is here: https://github.com/BlueBrain/HighFive/blob/master/include/highfive/H5DataType.hpp#L242 |
Codecov Report
@@ Coverage Diff @@
## master #796 +/- ##
==========================================
+ Coverage 84.21% 84.25% +0.04%
==========================================
Files 68 68
Lines 4776 4795 +19
==========================================
+ Hits 4022 4040 +18
- Misses 754 755 +1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Description
HighFive provides the ability to commit datatypes to files with CompoundType::commit for example but does not provide the ability to open the previously commited datatypes.
This PR enhance the Node Traits to get the datatype stored in that node.
How to test this?
cmake .. make -j8 make test
Test System
-OS: Ubuntu 22.04.2 LTS (WSL)
-Compiler: gcc 11.3.0
-Dependency versions: hdf5 1.10.7