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

Fix arguments for setting pdfindex for toys in tutorial #844

Merged

Conversation

kcormi
Copy link
Collaborator

@kcormi kcormi commented Jun 29, 2023

Fixing a small mistake in the parametric fit tutorial documentation about using RooMultiPDF

As pointed out on cms talk, the current documentation says to use

--setParameters pdfindex=i --freezeParameters pdfindex=j

when doing the fit, to test the fit using a different pdf than was used to generate the toys. But the second part is not a valid argument, it should only take the parameter name, not a value.

@kcormi kcormi added the documentation Updates for the documentation label Jun 29, 2023
@anigamova anigamova merged commit a849c26 into cms-analysis:main Jul 25, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Updates for the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants