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.

Sign up for GitHub

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

Jump to bottom

add option to measure_optics for selecting BPM pairing for coupling measurement #333

Open
awegsche opened this issue Sep 30, 2021 · 0 comments
Open

add option to measure_optics for selecting BPM pairing for coupling measurement #333

awegsche opened this issue Sep 30, 2021 · 0 comments
Labels
Priority: Low Work on this if you have some spare time. Type: Feature A (suggetion for a) new feature or enhancement in functionality.

Comments

@awegsche
Copy link
Contributor

awegsche commented Sep 30, 2021 •
edited
Loading

The new coupling code switches to a sophisticated method for selecting the BPM pairings for the momentum reconstruction as opposed to either just nearest neighbours (original method) or constantly jumping one (latest in GetLLM).

We should add a switch to select the method for

  • testing the implementation of the algorithm (in regression tests against old codes)
  • being able to quickly compare to archived measurements.
@awegsche awegsche added Type: Feature A (suggetion for a) new feature or enhancement in functionality. Priority: Low Work on this if you have some spare time. labels Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority: Low Work on this if you have some spare time. Type: Feature A (suggetion for a) new feature or enhancement in functionality.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fsoubelet @awegsche @pylhctokens