Add small fixes for CORDS tests

Desbordante · Oct 17, 2024 · d3069e2 · d3069e2
1 parent 5ba3492
commit d3069e2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/core/algorithms/fd/sfd/cords.cpp b/src/core/algorithms/fd/sfd/cords.cpp
@@ -75,7 +75,8 @@ void Cords::MakeExecuteOptsAvailableFDInternal() {
     using namespace config::names;
     MakeOptionsAvailable({kOnlySFD, kMinCard, kMaxDiffValsProportion, kMinSFDStrengthMeasure,
                           kMinSkewThreshold, kMinStructuralZeroesAmount,
-                          kMaxFalsePositiveProbability, kDelta, kMaxAmountOfCategories});
+                          kMaxFalsePositiveProbability, kDelta, kMaxAmountOfCategories,
+                          kFixedSample});
 }
 
 void Cords::ResetStateFd() {

diff --git a/src/core/algorithms/fd/sfd/sample.cpp b/src/core/algorithms/fd/sfd/sample.cpp
@@ -24,7 +24,7 @@ Sample::Sample(bool fixed_sample, unsigned long long sample_size, model::TupleIn
     std::unordered_set<std::string> map_cardinality;
 
     for (model::ColumnIndex i = 0; i < sample_size; i++) {
-        model::TupleIndex row = (fixed_sample) ? i : distribution(gen);
+        model::TupleIndex row = (fixed_sample) ? i % rows : distribution(gen);
 
         row_indices_.push_back(row);
         map_lhs.insert(data[lhs].GetDataAsString(row));