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

Closed orbit in momentum aperture #637

Closed
oscarxblanco opened this issue Jul 28, 2023 · 2 comments
Closed

Closed orbit in momentum aperture #637

oscarxblanco opened this issue Jul 28, 2023 · 2 comments

Comments

@oscarxblanco
Copy link
Contributor

Dear all,

in get_momentum_acceptance pyat calculates the closed orbit on each reference point.
For a lattice with errors the closed orbit guess around zeros might not converge.

It would be better to either calculate the closed orbit at the start of the ring and propagate the coordinates througth out the accelerator up to the reference point, or to store the orbit in all reference points and use it later on in the routine.

I have tried the second approach by using the parameter offset in get_acceptance to pass the closed orbit, however, it seems to consider a single 6D point offset.

Is there any way to pass the closed orbit to momentum aperture ? or is offset able to work with multiple reference points ?

Best regards,
o

@swhite2401
Copy link
Contributor

@oscarxblanco get_1d_acceptance that is used in get_momentum_acceptance has no offset parameter.
In any case you are correct, the offset is not correctly handled in the presence of multiple refpts so this would need to be improved.

I can propose the following changes:
-adapt offset in get_acceptance to handle multiple refpts
-introduce offset in get_1d_acceptance

In this way you should be able to pre-compute the closed orbit at your observations points and pass it to the function function through the offset argument.

Would that work for you?

@oscarxblanco
Copy link
Contributor Author

Yes, this sounds great

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants