-
Notifications
You must be signed in to change notification settings - Fork 22
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
Reference Particle: Store Charge & Mass #229
Conversation
f27b206
to
587b8c2
Compare
Store the mass and charge of the reference particle. This simpliifes init logic and is useful for later I/O tasks of meta data.
e67d523
to
0dd09fd
Compare
dec4453
to
9ab0898
Compare
9ab0898
to
cb56292
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty nice. I like that the reference particle parameters are set first now. 👍
} | ||
else { | ||
qm = 0.0; | ||
else { // default to electron |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't I be able to set up muons, for instance, if I provided a name and qe
and massE
for them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we did not add that case yet but could issue a warning, for instance.
Store the mass and charge of the reference particle.
This simplifies init logic and is useful for later I/O tasks of meta data.
Needed for #162 and #214.