-
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
NetCDF backend #33
Comments
@lge0303 is interested in providing an implementation. Thank you! ✨ I did sent you a GitHub invite to the openPMD organization so I can assign you in issues :) |
@lge0303 how are you doing? do you need anything from us for the implementation? Please feel free to open issues if you experience any installation issues or something is not well documented or similar. We are happy to help! |
Axel,
Thanks for asking. Sorry I’m busy doing other projects in the past weeks, will pick up the NetCDF backends implementation later this week. Will let you know if I have problems.
Bests!
Lixin
|
Hi Axel,
I’m implementing the I/O operations in the IOHandlerImpl following the Style Guide. What editor tool you use to easily follow the code Style Guide? Any suggestions?
Thanks!
Lixin
|
I usually just use netbeans. We have no automated enforcement/formatting of C++ code with e.g. Best to try to write in the same style as the other parts are already written, ideal patches (pull requests) should look as if they were all from one (virtual) author which simplifies reading for all. |
Hi @lge0303. Great to hear you're progressing on this. If you run into any implementation issues, don't hesitate to contact me. My daily driver for this project from the start has been CLion. If you're in academia or developing an open source project, you can even get their products for free. |
Axel,
Which make file should I modify to add netcdf backends to openPMD-api? I modified CMakeLists.txt under openPMD-api directory, the operations are the same as for ADIOS. What else should I do?
Thanks!
Lixin
|
Hi Lixin,
The other file for building/installing is openPMDConfig.cmake.in
Axel
|
Hi @lge0303. Thanks for taking your time to contribute to this project! To better understand what your current status is and to make it easier for us to assist you, I want to ask you to open a pull request. You can create the PR here. For reference, this is what the recent ADIOS backend pull request looked like. |
If you have a very quick question, we also have a gitter channel right here. |
Fabian,
Yes, I’ll do a pull request for NetCDF backends development.
Thanks for your suggestions!
Lixin
|
Hi Axel,
Thanks!
Lixin
|
@lge0303 Hi Lixin, how are you doing? |
Hi Axel,
Thanks for asking.
I forked a netcdf backends branch, did some development, will pull request soon.
Bests!
Lixin
|
@lge0303 Hi and a happy new year! Are you still working on this? |
Axel, Happy New year!
I’m working on the write Electromagnetic Field based on Finite Element to E/M field on finite difference grids in openPMD format, which will be read in by IMPACT - a beam dynamic software. So currently I’m not working on NetCDF backend of OpenPMD-api. I’m not sure when I could pick it up, sorry about it.
Lixin
|
A NetCDF backend would be useful for FermiLab & SLAC codes.
Note: NetCDF4 can use HDF5 underneath.
Note: NetCDF also supports serial & parallel I/O.
Resources: how to write a backend
The text was updated successfully, but these errors were encountered: