Draft: Updates for ATLAS+CMS (!116) · Merge requests · Alkaid Cheng / QuickStats · GitLab

Yizhou Cai requested to merge caiyi/quickstats:yizhousBranch into master Jan 10, 2024

The change at L101 of quickstats/components/asimov_generator.py is because that when calling this class at https://gitlab.cern.ch/clcheng/quickstats/-/blob/master/quickstats/clis/stat_tools.py#L87 the input variables are more.

The additional modification of pdf where self.pdf = ROOT.RooSimultaneousOpt(self.pdf) is discussed in s14 here https://indico.cern.ch/event/1329773/contributions/5597659/attachments/2723710/4732877/ATLASCMS_HHcomb_280923.pdf .

The new_pdfIndex judgement is to fix an issue that the same pdfIndex could increase the minimize runtime a lot because it is already in the minimum initially before https://gitlab.cern.ch/clcheng/quickstats/-/blob/master/quickstats/components/extended_minimizer.py?ref_type=heads#L733 .