Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[WIP] openPMD: Group-Based Encoding #222

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

[WIP] openPMD: Group-Based Encoding #222

wants to merge 1 commit into from

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Jan 22, 2024 •
edited
Loading

Since a LASY file usually describes one data snapshot, the openPMD groupBased encoding is more suitable and less confusing in file naming.

https://github.com/openPMD/openPMD-standard/blob/1.1.0/STANDARD.md#iterations-and-time-series

  • default to groupBased and add an option to change it if desired to fileBased (as in WarpX)
  • update FBPIC
  • update WarpX
  • update HiPACE++
  • update Wake-T
  • double check openPMD-viewer with both backends (openPMD-api and h5py)

@ax3l
openPMD: Group-Based Encoding
ad6031f 
Since a LASY file usually describes one data snapshot, the
openPMD `groupBased` encoding is more suitable and less confusing
in file naming.
@ax3l ax3l added design Raise discussion on LASY architecture and design cleaning Code cleaning labels Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cleaning Code cleaning design Raise discussion on LASY architecture and design
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ax3l