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.

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

Move checksum library to a separate repo? #877

Open
MaxThevenet opened this issue Mar 1, 2023 · 0 comments
Open

Move checksum library to a separate repo? #877

MaxThevenet opened this issue Mar 1, 2023 · 0 comments

Comments

@MaxThevenet
Copy link
Member

WarpX and HiPACE++ currently have their own copy of our checksum library: https://github.com/Hi-PACE/hipace/tree/development/tests/checksum. They diverged slightly, but nothing hard to fix. We briefly discussed putting this on a separate repo, to make sure contributions to it are beneficial to both codes easily. Also, it would make sense if other codes were interested in using it. I open this issue to start a discussion, and have a few questions to help with the decision:

  1. Do we want to support multiple output format, or just openPMD? We currently support plotfile and openPMD.
  2. (somehow related to 1) Would other codes consider using it? e.g. AMReX @WeiqunZhang @atmyers, FBPIC @RemiLehe or Wake-T @AngelFP?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant