Ryan M. L. McFadden authored 3 months ago and suter_a committed 3 months ago

Explicit passing of MnStrategy during minimizer initialization

Approved-by: Suter Andreas

This patch fixes compilation errors that arise with recent ROOT versions (e.g., 6.34.02),
which require a ROOT::Minuit2::MnStrategy object to be passed when initializing the
ROOT::Minuit2::MnX (X = Migrad, Simplex, Minimize) minimizers (i.e., instead of the integer
representing the "strategy").

make the theme handling in musredit more flexible. At the same time adopted some part to more modern c++.

make sure that for non-muSR data errors are not = 0. Will set it to 1, and issue a warning. This prevents chisq to crash.

Ryan M. L. McFadden authored 7 months ago and suter_a committed 7 months ago

fix batch mode averaged output

Approved-by: Andreas Suter

closes issue #63

added the option to write the content of the MINUIT2.OUTPUT ascii file as yaml. This extends the previous commit from Ryan M.L. McFadden.

Ryan M. L. McFadden authored 10 months ago and suter_a committed 10 months ago

ignore all files generated from an in-repo build

Approved-by: Andreas Suter

Ryan M. L. McFadden authored 10 months ago and suter_a committed 10 months ago

write the fit results to an easy-to-read/parse yaml file

Approved-by: Andreas Suter

Ryan M. L. McFadden authored 10 months ago and suter_a committed 10 months ago

fix segfault

Approved-by: Andreas Suter

This patch adds routines for writing a comprehensive set of fit results (i.e., parameter values, parabolic errors, asymmetric errors, covariances, correlation coefficients, etc.) for an individual `.msr` file to an easy-to-read/parse `.yaml` file.

The main motivation for the code addition is to provide users with easy access to the fit's covariance matrix without the need for "extra" manual effort (e.g., parsing the contents of `MINUIT2.OUTPUT` or `MINUIT2.root`). The other fit quantities are also included for completeness.

Reading/accessing the output is trivial using, for example, the PyYAML Python library (https://github.com/yaml/pyyaml):

```python
import yaml

with open("2125_tf_histo.yaml", "r") as fh:
	results = yaml.load(fh, Loader=yaml.SafeLoader)

cov = results["covariance"]["Field_1"]["Sigma_1"]
```

Note: the naming conventions chosen for the blocks in the `.yaml` output closely follow those used by the iminuit Python library (https://github.com/scikit-hep/iminuit).

This patch provides a memory-safe alternative to the changes introduced in commit 418adfde, which causes a segfault when the "batch mode" flag is required (i.e., for ascii/graphic export).

Note: the program name (i.e., argv[0]) has been added the list arguments passed to ROOT's TApplication. This ensures that the TApplication name matches that of the program (see https://github.com/root-project/root/blob/542b98b2ccca760fd83117b750b89d81b8e9b926/core/base/src/TApplication.cxx#L179-L180).