-
Notifications
You must be signed in to change notification settings - Fork 35
Add dp to rdts #890
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
base: master
Are you sure you want to change the base?
Add dp to rdts #890
Conversation
Dear @swhite2401 and @lfarv , who could review this PR ? |
@oscarxblanco this looks fine to me, are the off-momentum RDTS the same as the on-momentum ones? High order terms appear no? |
@swhite2401 , @lfarv . >>> ring.avlinopt(refpts=at.All)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/home/oblanco/Documents/public/progs/pyenv/pyenvat/lib/python3.11/site-packages/at/lattice/utils.py", line 279, in wrapper
raise AtError('{0} needs "ring.is_6d" {1}'.format(
at.lattice.utils.AtError: avlinopt needs "ring.is_6d" False |
I thought I had fixed this by calling linopt6 in avlinopt, do you run the latest master version? |
By the way I discussed with @carmignani on that point and the implementation you propose is missing all the feed down effects from the off-energy orbit. |
@swhite2401 , yes, it is the latest master
|
oh, I see.
Am I missing something ? |
Yes I think this is about it |
Concerning the avlinopt calculation I changed the linopt function but did not remove the decorator... this explains the behavior you observe, it can be easily fixed |
This PR adds the energy offset as option to the calculation of the optics used in AT matlab
computeRDT
and pyatget_rdts
.