Skip to content

Commit

Permalink
fix perf test (#3253)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #3253

relaxing thresholds + allow equal, duh

Reviewed By: mlomeli1

Differential Revision: D53803000

fbshipit-source-id: 7b68386523d9a44a0cd8e099fc1d72b3ed7d5e43
  • Loading branch information
algoriddle authored and facebook-github-bot committed Feb 15, 2024
1 parent 6d00c41 commit 873b1bc
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/test_fastscan_perf.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ TEST(TestFastScan, knnVSrange) {
.count();

// we expect the perf of knn and range search
// to be similar, at least within a factor of 2
ASSERT_LT(range_time, knn_time * 2);
ASSERT_LT(knn_time, range_time * 2);
// to be similar, at least within a factor of 4
ASSERT_LE(range_time, knn_time * 4);
ASSERT_LE(knn_time, range_time * 4);
}

0 comments on commit 873b1bc

Please sign in to comment.