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

[Bug]: Beam direction possibly wrong for Beam 4/Beam 2 tracked backwards #475

Open
JoschD opened this issue Nov 28, 2024 · 1 comment
Open
Labels
Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Priority: Medium Work on this. Type: Bug Something isn't working as it should.

Comments

@JoschD
Copy link
Member

JoschD commented Nov 28, 2024

Bug Description

The opposite_direction flag is only used in harpy, but opposite direction is assumed later on in the optics analysis for Beam 2. This is fine for measurement data, but when working with tracking data model and turn-by-turn data can be in the same direction, even for Beam 2. This is not taken into account (in the accelerator class it is always set to -1).

Steps to Reproduce

Run optics analysis with Beam 4 tbt and model.

Possible Fix Implementation

Make opposite_direction an LHC accelerator-class parameter?

Also needs to be propagated then to the GUI.

@JoschD JoschD added Type: Bug Something isn't working as it should. Priority: Medium Work on this. Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. labels Nov 28, 2024
@JoschD
Copy link
Member Author

JoschD commented Nov 28, 2024

@Mael-Le-Garrec you have encountered this before, right? Any input?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Priority: Medium Work on this. Type: Bug Something isn't working as it should.
Projects
None yet
Development

No branches or pull requests

1 participant