diff options
author | Marc Glisse <marc.glisse@inria.fr> | 2021-04-16 22:49:55 +0200 |
---|---|---|
committer | Marc Glisse <marc.glisse@inria.fr> | 2021-04-17 18:52:37 +0200 |
commit | 9616e2b8f616366393bf0b74a76029ae8a95d77a (patch) | |
tree | 1a257a9d4986c844a5f3a099fb7f602a048eac19 /src/Subsampling/test/test_choose_n_farthest_points.cpp | |
parent | 7e05e915adc1be285e04eb00d3ab7ba1b797f38d (diff) |
Rewrite choose_n_farthest_points
Introduce an indirection to help with points with multiplicity.
Limit the use of magic values like 0 and infinity.
Diffstat (limited to 'src/Subsampling/test/test_choose_n_farthest_points.cpp')
-rw-r--r-- | src/Subsampling/test/test_choose_n_farthest_points.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/Subsampling/test/test_choose_n_farthest_points.cpp b/src/Subsampling/test/test_choose_n_farthest_points.cpp index 94793295..c384c61b 100644 --- a/src/Subsampling/test/test_choose_n_farthest_points.cpp +++ b/src/Subsampling/test/test_choose_n_farthest_points.cpp @@ -102,11 +102,12 @@ BOOST_AUTO_TEST_CASE_TEMPLATE(test_choose_farthest_point_limits, Kernel, list_of BOOST_CHECK(distances[1] == 1); landmarks.clear(); distances.clear(); - // Ignore duplicated points + // Accept duplicated points points.emplace_back(point.begin(), point.end()); Gudhi::subsampling::choose_n_farthest_points(d, points, -1, -1, std::back_inserter(landmarks), std::back_inserter(distances)); - BOOST_CHECK(landmarks.size() == 2 && distances.size() == 2); + BOOST_CHECK(landmarks.size() == 3 && distances.size() == 3); BOOST_CHECK(distances[0] == std::numeric_limits<FT>::infinity()); BOOST_CHECK(distances[1] == 1); + BOOST_CHECK(distances[2] == 0); landmarks.clear(); distances.clear(); } |