-
Notifications
You must be signed in to change notification settings - Fork 51
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
HDF5: Explicit control over chunking #1591
Conversation
I see there are two styles of json for series and dataset
any chance to consolidate? |
That is TOML, and both forms are equivalent in TOML. The JSON form would be |
Oh, I see. Was wondering.
|
2fb6a0a
to
799d465
Compare
Still need to filter out the warnings better
This reverts commit f00baa7.
1b0062d
to
980403b
Compare
src/IO/HDF5/HDF5IOHandler.cpp
Outdated
//! @todo allow overwrite with user-provided chunk size | ||
// for( auto const& val : parameters.chunkSize ) | ||
// chunk_dims.push_back(static_cast< hsize_t >(val)); |
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.
@franzpoeschel can this todo now be removed?
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.
yep, this todo is now fixed
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.
Looks great!
Please see last inline comment and then feel free to merge :)
Thanks! I removed the old todos |
TODO: