Skip to content

Commit

Permalink
Update Combine.cc
Browse files Browse the repository at this point in the history
There is no `-S 0` option anymore so removed suggestion to use it
  • Loading branch information
nucleosynthesis authored Apr 29, 2024
1 parent f23b469 commit e791492
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/Combine.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1018,7 +1018,7 @@ void Combine::run(TString hlfFile, const std::string &dataset, double &limit, do
allFloatingParameters.remove(*mc->GetParametersOfInterest());
int nFloatingNonPoiParameters = utils::countFloating(allFloatingParameters);
if (nFloatingNonPoiParameters && !toysNoSystematics_ && (readToysFromHere == 0)) {
if (nuisances == 0) throw std::logic_error("Running with systematic variation in toys enabled, but I found floating parameters (which are not POIs) and no constraint terms have been defined in the datacard. If this is ok, re-run with -S 0");
if (nuisances == 0) throw std::logic_error("Running with systematic variation in toys enabled, but I found floating parameters (which are not POIs) and no constraint terms have been defined in the datacard.");
nuisancePdf.reset(utils::makeNuisancePdf(expectSignal_ || setPhysicsModelParameterExpression_ != "" || noMCbonly_ ? *mc : *mc_bonly));
if (toysFrequentist_) {
if (mc->GetGlobalObservables() == 0) throw std::logic_error("Cannot use toysFrequentist without global observables");
Expand Down

0 comments on commit e791492

Please sign in to comment.