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

ED-PIC: Add Pusher Values #172

Merged
merged 2 commits into from
Jan 30, 2018
Merged

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Jan 23, 2018

Adds new allowed pusher values for classical radiation reaktion and diagnostics to the particlePush attribute in the ED-PIC extensions.

Implements issues: #132 #139 #145

Description

Adds the following new pushers:

  • free-streaming (constantly moving with initial momentum)
  • LLRK4 (reduced Laundau-Lifshitz pusher via RK4 and classical radiation reaction, doi:10.1016/j.cpc.2016.04.002)
  • none (static particles such as probes)

none is a special case of streaming but probably still worth to be added to be explicit.

Affected Components

  • EXT: ED-PIC

Logic Changes

None.

Writer Changes

Writers can now switch from other for these three pushers to the newly specified values.

Reader Changes

Just more allowed values are known now which were previously undefined as other.

Data Converter

Per-application possible but not needed. Old files are still forward compatible to this change.

Adds new allowed pusher values to the `particlePush` attribute in
the ED-PIC extensions.

- `streaming` (constantly moving with initial momentum)
- `LLRK4` (reduced Laundau-Lifshitz pusher via RK4 and classical
           radiation reaction,
           [doi:10.1016/j.cpc.2016.04.002](https://dx.doi.org/10.1016/j.cpc.2016.04.002))
- `none` (static particles such as probes)
@ax3l ax3l added minor change backwards-compatible change EXT: ED-PIC electro-magnetic particle-in-cell extrension labels Jan 23, 2018
@ax3l ax3l requested a review from RemiLehe January 23, 2018 13:22
@ax3l ax3l mentioned this pull request Jan 23, 2018
@RemiLehe
Copy link
Member

Instead of streaming, I would maybe suggest ballistic. Any opinion?

@ax3l
Copy link
Member Author

ax3l commented Jan 23, 2018

Ballistic sounds more physical, I like that.

I just new "free streaming" as a term used in PWFA sims, but I can change it to ballistic :)

@ax3l ax3l changed the title ED-PIC: Add Pusher Values [WIP] ED-PIC: Add Pusher Values Jan 23, 2018
@ax3l
Copy link
Member Author

ax3l commented Jan 25, 2018

@RemiLehe thinking about it and reading e.g. the default wikipedia articles on it, yes ballistic means "free to move" but usually under forces such as the gravitational force. I think we want to avoid that impression since e.g. a "free-streaming" electron beam will not change its trajectory in background electro-magnetic fields in idealized driver models that use such a pusher.

@ax3l ax3l changed the title [WIP] ED-PIC: Add Pusher Values ED-PIC: Add Pusher Values Jan 25, 2018
@RemiLehe
Copy link
Member

OK, good point. Then maybe we can change streaming to free-streaming. Does that sound ok?

@ax3l
Copy link
Member Author

ax3l commented Jan 25, 2018

Good idea, that's more explicit!
or just free or constant pusher?

@RemiLehe
Copy link
Member

I think free-streaming is the most explicit.

Change naming to be more explicit.
@ax3l ax3l merged commit bab1aa3 into openPMD:upcoming-1.1.0 Jan 30, 2018
@ax3l ax3l deleted the topic-picNewPushers branch January 30, 2018 10:04
This was referenced Jan 30, 2018
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 minor change backwards-compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants