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

png2gas: adjust for openPMD 1.0.0 #1446

Open
ax3l opened this issue Apr 27, 2016 · 5 comments
Open

png2gas: adjust for openPMD 1.0.0 #1446

ax3l opened this issue Apr 27, 2016 · 5 comments
Labels
component: tools scripts, python libs and CMake refactoring code change to improve performance or to unify a concept but does not change public API
Milestone
Future

Comments

@ax3l
Copy link
Member

ax3l commented Apr 27, 2016

The png2gas tools should be adjusted to write openPMD 1.0.0 files (and read those in PIConGPU examples).
@ax3l ax3l added component: tools scripts, python libs and CMake refactoring code change to improve performance or to unify a concept but does not change public API labels Apr 27, 2016
@ax3l ax3l added this to the Open Beta milestone Apr 27, 2016
@ax3l ax3l modified the milestones: Future, 0.2.0: Open Beta Nov 2, 2016
@ax3l
Copy link
Member Author

ax3l commented Jul 31, 2017

@n01r please be aware of this issue that png2gas might need slight modifications now

@n01r
Copy link
Member

n01r commented Jul 31, 2017

Thanks! So far it seemed that it would suffice for the needs since if the number of cells in each direction is equal to the pixel dimensions. Reading worked as intended and running worked as well.

@ax3l
Copy link
Member Author

ax3l commented Jul 31, 2017

excellent. yes it does not really need the openPMD meta information yet, we read the data rather simple. feel free to report if you encounter any issues and make sure to use PNGwrite 0.6.0 due to the 8bit read bugs documented here: https://github.com/pngwriter/pngwriter/blob/0.6.0/CHANGELOG.md

@n01r
Copy link
Member

n01r commented Jul 31, 2017

Yep, had to build PNGwriter 0.6.0 since it was required and it was not available on the cluster.

@ax3l
Copy link
Member Author

ax3l commented Jul 31, 2017 •
edited
Loading

Already requested PNGwriter 0.6.0 to be installed a few hours ago, will take a few days due to vacations but building your own is fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: tools scripts, python libs and CMake refactoring code change to improve performance or to unify a concept but does not change public API
Projects
None yet
Milestone
Future
Development

No branches or pull requests
2 participants
@ax3l @n01r