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

Default particle #367

Merged
merged 1 commit into from
Feb 14, 2022
Merged

Default particle #367

merged 1 commit into from
Feb 14, 2022

Conversation

lfarv
Copy link
Contributor

@lfarv lfarv commented Feb 13, 2022

The default particle is set back to 'relativistic', meaning "relativistic electron", and its definition is modified as follows:

  • its rest energy is zero,
  • its charge is -1,
  • its relativistic β is 1,
  • its relativistic γ, as reported by the Lattice.gamma property is the one of an electron.

This is physically inconsistent (see discussion here), but it describes the historical behaviour of AT.
It allows to generalise the use of the lattice properties Lattice.gamma and Lattice.beta everywhere, while being consistent with tracking.

In addition, the energy and particle properties are made available to the C integrators, in preparation for non-relativistic tracking. The "global" parameters transmitted to the C integrators are now:

  • ring length,
  • ring revolution time,
  • turn number,
  • energy,
  • particle rest energy,
  • particle charge,
  • s coordinate

The new parameters must not be used until the equivalent is made for Matlab!

@lfarv lfarv added the Python For python AT code label Feb 14, 2022
Copy link
Contributor

@swhite2401 swhite2401 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok for me

@lfarv lfarv merged commit 604f849 into master Feb 14, 2022
@lfarv lfarv deleted the energy_particle_in_C branch February 14, 2022 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Python For python AT code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants