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

LSA knob to MAD-X script converter #100

Merged
merged 28 commits into from
May 23, 2022
Merged

LSA knob to MAD-X script converter #100

merged 28 commits into from
May 23, 2022

Conversation

fsoubelet
Copy link
Member

@fsoubelet fsoubelet commented May 20, 2022

Adds a script which can take an LSA knob and an optics it is defined for to create both a definition TFS file and a MAD-X script which will reproduce the knob in simulations.

The script can also be fed a text file with many knobs definitions as well as their trim values, and run for all of these knobs. See examples in the module docstring.

@fsoubelet fsoubelet self-assigned this May 20, 2022
@fsoubelet fsoubelet requested review from JoschD and mihofer May 20, 2022 15:34
@JoschD
Copy link
Member

JoschD commented May 20, 2022

why is your diff coverage so low, with all of these tests? xD

@fsoubelet fsoubelet requested a review from JoschD May 22, 2022 06:38
mihofer
mihofer previously approved these changes May 23, 2022
@fsoubelet fsoubelet merged commit 7fb7ae0 into master May 23, 2022
@fsoubelet fsoubelet deleted the lsa_knob branch May 23, 2022 15:23
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

Successfully merging this pull request may close these issues.

3 participants