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

Update runningthetool.md #885

Merged
merged 2 commits into from
Dec 20, 2023
Merged

Update runningthetool.md #885

merged 2 commits into from
Dec 20, 2023

Conversation

nucleosynthesis
Copy link
Contributor

Add explanation as to why repeated commands wont give identical results.

Also moved branch discussion out of info box.

Add explanation as to why repeated commands wont give identical results.

Also moved branch discussion out of info box.
Copy link
Collaborator

@kcormi kcormi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this looks good, just spotted one grammatical error that we might as well fix while we're at it.

@@ -152,8 +152,11 @@ The option `-t` is used to tell <span style="font-variant:small-caps;">Combine</

The output file will contain the toys (as `RooDataSets` for the observables, including global observables) in the **toys** directory if the option `--saveToys` is provided. If you include this option, the `limit` TTree in the output will have an entry corresponding to the state of the POI used for the generation of the toy, with the value of **`quantileExpected`** set to **-2**.

!!! info
The branches that are created by methods like `MultiDimFit` *will not* show the values used to generate the toy. If you also want the TTree to show the values of the POIs used to generate to toy, you should add additional branches using the `--trackParameters` option as described in the [common command-line options](#common-command-line-options) section above. These branches will behave as expected when adding the option `--saveToys`.
The branches that are created by methods like `MultiDimFit` *will not* show the values used to generate the toy. If you also want the TTree to show the values of the POIs used to generate to toy, you should add additional branches using the `--trackParameters` option as described in the [common command-line options](#common-command-line-options) section above. These branches will behave as expected when adding the option `--saveToys`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"used to generate to toy" --> used to generate the toy.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! fixed now

@kcormi kcormi merged commit 2763577 into main Dec 20, 2023
6 checks passed
@nucleosynthesis nucleosynthesis deleted the nucleosynthesis-patch-3 branch February 15, 2024 13:57
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.

2 participants