Skip to content

Commit

Permalink
dont parallelise spectrum sampling, added some comments
Browse files Browse the repository at this point in the history
  • Loading branch information
karlnapf authored and lambday committed Jun 28, 2016
1 parent 4eec2a4 commit 1fde8b2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/shogun/statistical_testing/QuadraticTimeMMD.cpp
Expand Up @@ -95,6 +95,7 @@ float64_t CQuadraticTimeMMD::compute_p_value(float64_t statistic)
break;
}
default:
// handles sampled null distributions
result = CHypothesisTest::compute_p_value(statistic);
break;
}
Expand All @@ -114,6 +115,7 @@ float64_t CQuadraticTimeMMD::compute_threshold(float64_t alpha)
break;
}
default:
// handles samples null distributions
result = CHypothesisTest::compute_threshold(alpha);
break;
}
Expand Down Expand Up @@ -174,8 +176,6 @@ SGVector<float64_t> CQuadraticTimeMMD::sample_null()
index_t max_num_eigenvalues = eigen_solver.eigenvalues().rows();

/* finally, sample from null distribution */

#pragma omp parallel for
for (auto i = 0; i < null_samples.vlen; ++i)
{
float64_t null_sample = 0;
Expand Down

0 comments on commit 1fde8b2

Please sign in to comment.