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

Add synergia example diagnostics output #21

Open
wants to merge 2 commits into
base: draft
Choose a base branch
from

Conversation

s-sajid-ali
Copy link
Member

No description provided.

README.md Outdated Show resolved Hide resolved
@ax3l ax3l self-assigned this Apr 17, 2024
@ax3l
Copy link
Member

ax3l commented Apr 17, 2024

@s-sajid-ali Thank you for the PR! I took a look at the file and albeit valid openPMD, it does not seem to contain particles. Can you double check?

$ h5ls -r synergia-fodo-diag.h5 
/                        Group
/data                    Group
/data/0                  Group
/data/1                  Group
/data/10                 Group
/data/11                 Group
/data/12                 Group
...
$ openpmd-ls synergia-fodo-diag.h5 
openPMD series: synergia-fodo-diag
openPMD standard: 1.1.0
openPMD extensions: 0

data author: unknown
data created: 2024-03-29 13:59:33 -0500
data backend: HDF5
generating machine: unknown
generating software: synergia3 (version: 2023.4.12-81c96e4d6)
generating software dependencies: unknown

number of iterations: 26 (groupBased)
  all iterations: 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 

number of meshes: 0

number of particle species: 0

@s-sajid-ali
Copy link
Member Author

Yes it does not contain any particles, it is the output of custom diagnostics as defined here:
https://github.com/fnalacceleratormodeling/synergia2/blob/1a85586989e545da5277a96131facc12d8b0d952/src/synergia/bunch/diagnostics_full2.cc#L60-L86

This was one of the files that the updater failed on before openPMD/openPMD-updater#10. With the updates there it simply notes that data is not present and only updates the top level attributes.

@s-sajid-ali
Copy link
Member Author

I can add file containing particles from the same run if that helps set the context for this file.

@ax3l
Copy link
Member

ax3l commented Apr 17, 2024

I see. I would call this file something like _empty.h5 then maybe?

Let's also add a valid Synergia file that is a bit more instructive/useful for people to play with (vis, analysis, tests)?

@s-sajid-ali s-sajid-ali force-pushed the synergia_diag_parts branch 3 times, most recently from a3be477 to c8f2f00 Compare April 19, 2024 16:16
@s-sajid-ali
Copy link
Member Author

Updated with more files from running the example. Let me know it looks good now.

Sajid Ali added 2 commits April 19, 2024 11:19
	modified:   README.md
	deleted:    synergia-fodo-diag.h5
	new file:   synergia-fodo.tar.gz
@s-sajid-ali s-sajid-ali force-pushed the synergia_diag_parts branch from c8f2f00 to ae2db7a Compare April 19, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants