-
Notifications
You must be signed in to change notification settings - Fork 0
Updates to variable-based encoding #1
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
Updates to variable-based encoding #1
Conversation
FORMAT_ADIOS.md
Outdated
| double /data/meshes/E/x/unitSI 10*scalar | ||
| double /data/meshes/E/unitDimension attr = {0, 0, 0, 0, 0, 0, 0} | ||
| int32_t /data/meshes/E/x {1000} | ||
| double /data/meshes/E/x/position attr = 0 |
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.
position should in 3D be 3 doubles
STANDARD.md
Outdated
|
|
||
| Notes: | ||
|
|
||
| * In implementations without support for IO steps, the variable-based encoding of iterations may still be used for storage of a single iteration. |
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.
In these points, the new wording using IO steps might be confusing?
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.
I've reworded that to support for storing multiple versions of datasets/attributes, which I think is more explicit anyway.
Co-authored-by: Axel Huebl <[email protected]>
Make the example more realistic, also minor fixes
snapshotattribute as in Mapping between ADIOS steps and openPMD iterations openPMD/openPMD-api#949