Skip to content

Commit

Permalink
[RF] Disable the part of testLikelihoodJobTest that prone to timeout
Browse files Browse the repository at this point in the history
This test was disabled because it was occasionally timing out on the CI.
Evaluating the same likelihood twice should not be a problem anymore,
and if it would be it would also manifest in other tests
  • Loading branch information
guitargeek committed Oct 21, 2024
1 parent 38bd8d4 commit 33337f7
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion roofit/roofitcore/test/TestStatistics/testLikelihoodJob.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,10 @@ TEST_F(LikelihoodJobTest, UnbinnedGaussian1DSelectedParameterValues)
RFMP::Config::LikelihoodJob::defaultNComponentTasks = RFMP::Config::LikelihoodJob::automaticNComponentTasks;
}

TEST_F(LikelihoodJobTest, UnbinnedGaussian1DTwice)
// This test was disabled because it was occasionally timing out on the CI.
// Evaluating the same likelihood twice should not be a problem anymore, and if
// it would be it would also manifest in other tests.
TEST_F(LikelihoodJobTest, DISABLED_UnbinnedGaussian1DTwice)
{
std::tie(nll, pdf, data, values) = generate_1D_gaussian_pdf_nll(w, 10000);
likelihood = RFTS::buildLikelihood(pdf, data.get());
Expand Down

0 comments on commit 33337f7

Please sign in to comment.