-
Notifications
You must be signed in to change notification settings - Fork 89
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
Support for DSTs? #225
Comments
I have a gut feeling this would be a major change, but you should prototype this to find out. Could be some other approach to dynamic types would be preferable instead of trying to fit this into the existing API |
DST cannot impl |
I'm working on extending the APIs: https://github.com/Berrysoft/hdf5-ext/tree/master/hdf5-dst |
@Berrysoft I know this issue is a little dated, but I was wondering how well your approach worked? This feature would be very useful for me, and I might be able to contribute some time to it if any more work is needed to integrate it. |
@watsaig You can try the repo (hdf5-ext) I linked above. I use it in production:) However I forgot to publish them, and the docs are not complete. Will publish them ASAP. If you would like my approach, we can discuss in my repo:) |
Great to hear, and thanks for the quick reply! |
I recently need to write some DST (dynamically sized type) data to HDF5. The size of a DST struct is dynamically known, and also fixed because once determined, it won't change anymore. However, I know that DST in rust is not that easy to use, and I don't ask for many changes.
I would just suggest that changing
H5Type::type_descriptor()
toH5Type::type_descriptor(&self)
. As the size of a DST struct is dynamically known, we can get the size with&self
, and therefore can construct an instance ofTypeDescriptor
.Although this is a small suggestion, it may require a lot changes (I haven't tried). Would you consider such change? If you like it, I'd like to open a PR and try to implement it.
The text was updated successfully, but these errors were encountered: