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

Change name of best-knowledge madx file. #331

Open
JoschD opened this issue Sep 28, 2021 · 3 comments
Open

Change name of best-knowledge madx file. #331

JoschD opened this issue Sep 28, 2021 · 3 comments
Labels
Enhancement New feature or request Feature

Comments

@JoschD
Copy link
Member

JoschD commented Sep 28, 2021

job.create_model.madx is overwritten by best-knowledge model creator (the two creators are called by the GUI in a row).
Maybe should be job.create_bestknowledge_model.madx (see other model creators in python code on how to easily overwrite)

@JoschD JoschD added Enhancement New feature or request Feature labels Sep 28, 2021
@JoschD
Copy link
Member Author

JoschD commented Sep 28, 2021

Related: is the best knowledge model even used in omc3?

  • yes! (please check)

@awegsche
Copy link
Contributor

awegsche commented Oct 5, 2021

yes, beta_from_phase for example uses the best knowledge model for its model phase advances:

def _try_best_model(meas_input):
if meas_input.accelerator.model_best_knowledge is None:
LOGGER.debug("No best knowledge model - using the normal one.")
return meas_input.accelerator.model
return meas_input.accelerator.model_best_knowledge

which is ultimately used in lines 84 und 85 of

def n_bpm_method(meas_input, phase, plane, meas_and_mdl_tunes):

@awegsche
Copy link
Contributor

awegsche commented Oct 5, 2021

In general it should be used by every model-dependent algorithm like beta_from_phase and both AC-dipole compensation methods (especially Ryoichi's)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Feature
Projects
None yet
Development

No branches or pull requests

2 participants