It says to remove it, so why not?

This commit is contained in:
Ryan Curtin
2017-04-24 17:47:56 -04:00
parent 7f7561d9cb
commit 12cf0e7fcc
2 changed files with 6 additions and 14 deletions
+6 -10
View File
@@ -225,7 +225,6 @@ class RASearchRules
const double oldScore);
size_t NumDistComputations() { return numDistComputations; }
size_t NumEffectiveSamples()
{
if (numSamplesMade.n_elem == 0)
@@ -270,27 +269,24 @@ class RASearchRules
//! The instantiated metric.
MetricType& metric;
//! Whether to sample at leaves or just use all of it
//! Whether to sample at leaves or just use all of it.
bool sampleAtLeaves;
//! Whether to do exact computation on the first leaf before any sampling
//! Whether to do exact computation on the first leaf before any sampling.
bool firstLeafExact;
//! The limit on the largest node that can be approximated by sampling
//! The limit on the largest node that can be approximated by sampling.
size_t singleSampleLimit;
//! The minimum number of samples required per query
//! The minimum number of samples required per query.
size_t numSamplesReqd;
//! The number of samples made for every query
//! The number of samples made for every query.
arma::Col<size_t> numSamplesMade;
//! The sampling ratio
//! The sampling ratio.
double samplingRatio;
// TO REMOVE: just for testing
size_t numDistComputations;
//! If the query and reference set are identical, this is true.
bool sameSet;
@@ -64,7 +64,6 @@ RASearchRules(const arma::mat& referenceSet,
// Initialize some statistics to be collected during the search.
numSamplesMade = arma::zeros<arma::Col<size_t> >(querySet.n_cols);
numDistComputations = 0;
samplingRatio = (double) numSamplesReqd / (double) n;
Log::Info << "Minimum samples required per query: " << numSamplesReqd <<
@@ -135,9 +134,6 @@ double RASearchRules<SortPolicy, MetricType, TreeType>::BaseCase(
numSamplesMade[queryIndex]++;
// TO REMOVE
numDistComputations++;
return distance;
}