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

Weighting -> Weight #199

Open
ax3l opened this issue Sep 26, 2018 · 0 comments
Open

Weighting -> Weight #199

ax3l opened this issue Sep 26, 2018 · 0 comments
Labels
EXT: ED-PIC electro-magnetic particle-in-cell extrension major change non-backwards compatible change
Milestone

Comments

@ax3l
Copy link
Member

ax3l commented Sep 26, 2018

For consistency with most PIC codes, we should probably replace weighting with weight. Did a quick survey and most use weight indeed.

Related: ComputationalRadiationPhysics/picongpu#2708

@ax3l ax3l added major change non-backwards compatible change EXT: ED-PIC electro-magnetic particle-in-cell extrension labels Sep 26, 2018
@ax3l ax3l added this to the openPMD 2.X milestone Sep 26, 2018
DavidSagan added a commit to DavidSagan/openPMD-standard that referenced this issue Mar 6, 2019
ax3l pushed a commit that referenced this issue Oct 23, 2020
* Ported EXT_BeamPhysics.md from other branch.
* Changes made per discussion on openPMD.
* Minor update.
* Minor typo fixes.
* More reorg.
* Minor edits.
* Minor tweaks based upon comments from Axel and Remi.
* Fix Typo
* Very minor: Correct final "/" in names.
* Minor name correction.
* Minor mod
* Fix typo in text.
* Correct "attribute" in several places should have been "record".
* Added particleStatus record
* Update EXT_BeamPhysics.md
* Merged pxType and pyType into pxyType
* Simplified momentum
* Minor styalistic edits.
* Simplified position record.
* "time", "refTime", and "time-refTime" are now "time" and "timeOffset".
* Added external fields
* Changed "weighting" to "weight" as per opemPMD issue #199.
* more cleanup.
* Remove redundant chargeDead.
* Minor cleanup of particleStatus.
* Reinstated elementIndex and locationInElement which were accedentially deleted.
* Added numParticles attribute.
* Now the components of momentum and velocity are x,y,z the same as other vectors.
* Added chargeUnitSI attribute.
* Minor spelling fix.
* Change "charge" to "chargeState" as more accurate.
* Corrected some typos and added the fileType attribute.
* Removed reference energy and momentum records as well as total energy.
* Minor doc update.
* First pass at adding external mesh fields group.
* more...
* First pass at external fields definition.
* More mesh field devel.
* more...
* Minor mod
* minor change
* minor change
* more grid devel.
* Removed speed dataset.
* removed version number.
* cosmetic change.
* Cosmetic change.
* Small stylistic changes.
* Cosmetic change.
* Cosmetic change.
* Added note on complex storage.
* Changed standard on how complex numbers are stored.
* Minor description updates.
* Minor cleanup.
* Change from using Bx, By, Bz to MagneticField/x, y, z. And similarly for the electric field.
* Minor mod.
* Added particleID field.
* Modify "id" description.
* Removed paragraph on how to store complex numbers (this is now in the base standard).

Co-authored-by: Christopher Mayes <cem52@cornell.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EXT: ED-PIC electro-magnetic particle-in-cell extrension major change non-backwards compatible change
Projects
None yet
Development

No branches or pull requests

1 participant