cms-analysis / CombineHarvester

CMSSW package for the creation, editing and analysis of combine datacards and workspaces
cms-analysis.github.io/CombineHarvester/
15 stars 180 forks source link

combineTool.py -M Impacts cannot account for toy input #266

Open lcorcodilos opened 2 years ago

lcorcodilos commented 2 years ago

When using -t 1 --toysFile <filename>.root to load in a toy dataset, the Impacts algorithm will not account for the change in output file names that includes the seed.

For example, after --doInitialFit, the output will be a file that looks like higgsCombine_initialFit_Test.MultiDimFit.mH{m}.123456.root but the --doFits step will look for higgsCombine_initialFit_Test.MultiDimFit.mH{m}.root. I'm working around this by renaming the files between steps but I imagine this should be changed in lines like these [1].

[1] - https://github.com/cms-analysis/CombineHarvester/blob/0675d6ae83f694d836df1003b62f9919f1ca4556/CombineTools/python/combine/Impacts.py#L151-L154