Skip to content

Commit d463375

Browse files
guitargeekmaxgalli
authored andcommitted
Comment out the final timing printout
Printing out the time it took to run combine might seem useful, but it is problematic for automatic validation because the output is "random".
1 parent 45b037b commit d463375

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/Combine.cc

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ std::string Combine::parseRegex(std::string instr, const RooArgSet *nuisances, R
289289
}
290290

291291
bool Combine::mklimit(RooWorkspace *w, RooStats::ModelConfig *mc_s, RooStats::ModelConfig *mc_b, RooAbsData &data, double &limit, double &limitErr) {
292-
TStopwatch timer;
292+
//TStopwatch timer;
293293

294294
bool ret = false;
295295
try {
@@ -317,8 +317,13 @@ bool Combine::mklimit(RooWorkspace *w, RooStats::ModelConfig *mc_s, RooStats::Mo
317317
std::cout << " --- MODEL ---\n";
318318
w->Print("V");
319319
} */
320-
timer.Stop(); t_cpu_ = timer.CpuTime()/60.; t_real_ = timer.RealTime()/60.;
321-
printf("Done in %.2f min (cpu), %.2f min (real)\n", t_cpu_, t_real_);
320+
321+
// Printing out the time it took to run combine might seem useful, but it is
322+
// problematic for automatic validation because the output is "random".
323+
324+
// timer.Stop(); t_cpu_ = timer.CpuTime()/60.; t_real_ = timer.RealTime()/60.;
325+
// printf("Done in %.2f min (cpu), %.2f min (real)\n", t_cpu_, t_real_);
326+
322327
return ret;
323328
}
324329

0 commit comments

Comments
 (0)