Fix/nnsearch repro

[panquez]

No description provided.

[github-actions]

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v21.1.8) reports: 15 concern(s)

• src/geode/geometry/nn_search.cpp:53:9: warning: [modernize-use-nodiscard]

  function 'point' should be marked [[nodiscard]]

     53 |         const Point< dimension >& point( const index_t index ) const
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:58:9: warning: [modernize-use-nodiscard]

  function 'nb_points' should be marked [[nodiscard]]

     58 |         index_t nb_points() const
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:63:9: warning: [modernize-use-nodiscard]

  function 'neighbors' should be marked [[nodiscard]]

     63 |         std::vector< index_t > neighbors(
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:80:9: warning: [modernize-use-nodiscard]

  function 'neighbors' should be marked [[nodiscard]]

     80 |         std::vector< index_t > neighbors( const Point< dimension >& point,
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:116:9: warning: [modernize-use-nodiscard]

  function 'nearest_vertices' should be marked [[nodiscard]]

    116 |         std::vector< index_t > nearest_vertices(
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:122:50: warning: [readability-container-data-pointer]

  'data' should be used for accessing the data pointer instead of taking the address of the 0-th element

    122 |                 &copy( point )[0], nb_neighbors, &results[0], &distances[0] );
        |                                                  ^~~~~~~~~~~
        |                                                  results.data()

• src/geode/geometry/nn_search.cpp:122:63: warning: [readability-container-data-pointer]

  'data' should be used for accessing the data pointer instead of taking the address of the 0-th element

    122 |                 &copy( point )[0], nb_neighbors, &results[0], &distances[0] );
        |                                                               ^~~~~~~~~~~~~
        |                                                               distances.data()

• src/geode/geometry/nn_search.cpp:128:9: warning: [modernize-use-nodiscard]

  function 'colocated_index_mapping' should be marked [[nodiscard]]

    128 |         typename geode::NNSearch< dimension >::ColocatedInfo
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:128:9: warning: [modernize-use-nodiscard]

  function 'colocated_index_mapping<geode::Frame<2>>' should be marked [[nodiscard]]

    128 |         typename geode::NNSearch< dimension >::ColocatedInfo
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:128:9: warning: [modernize-use-nodiscard]

  function 'colocated_index_mapping<geode::Frame<3>>' should be marked [[nodiscard]]

    128 |         typename geode::NNSearch< dimension >::ColocatedInfo
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:129:13: warning: [readability-function-cognitive-complexity]

  function 'colocated_index_mapping' has cognitive complexity of 15 (threshold 10)

    129 |             colocated_index_mapping(
        |             ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:138:17: note: nesting level increased to 1
    138 |                 [&epsilon, &neighbor_vertices, this]( index_t point_id ) {
        |                 ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:143:13: note: +1, including nesting penalty of 0, nesting level increased to 1
    143 |             for( const auto point_id : Range{ nb_points } )
        |             ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:145:17: note: +2, including nesting penalty of 1, nesting level increased to 2
    145 |                 if( mapping[point_id] != NO_ID )
        |                 ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:149:17: note: +2, including nesting penalty of 1, nesting level increased to 2
    149 |                 for( const auto vertex_id : neighbor_vertices[point_id] )
        |                 ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:151:21: note: +3, including nesting penalty of 2, nesting level increased to 3
    151 |                     if( mapping[vertex_id] == NO_ID )
        |                     ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:159:13: note: +1, including nesting penalty of 0, nesting level increased to 1
    159 |             for( const auto point_id : Range{ nb_points } )
        |             ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:167:17: note: +2, including nesting penalty of 1, nesting level increased to 2
    167 |                 if( mapping[point_id] == point_id )
        |                 ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:175:13: note: +1, including nesting penalty of 0, nesting level increased to 1
    175 |             for( const auto point_id : Range{ nb_points } )
        |             ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:177:17: note: +2, including nesting penalty of 1, nesting level increased to 2
    177 |                 if( mapping[point_id] == point_id )
        |                 ^
  /__w/OpenGeode/OpenGeode/src/geode/geometry/nn_search.cpp:184:13: note: +1, including nesting penalty of 0, nesting level increased to 1
    184 |             for( const auto point_id : Range{ nb_points } )
        |             ^

• src/geode/geometry/nn_search.cpp:193:9: warning: [modernize-use-nodiscard]

  function 'copy' should be marked [[nodiscard]]

    193 |         std::array< double, dimension > copy(
        |         ^
        |         [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:196:13: warning: [cppcoreguidelines-pro-type-member-init]

  uninitialized record type: 'result'

    196 |             std::array< double, dimension > result;
        |             ^                                     
        |                                                   {}

• src/geode/geometry/nn_search.cpp:207:13: warning: [modernize-use-nodiscard]

  function 'kdtree_get_point_count' should be marked [[nodiscard]]

    207 |             size_t kdtree_get_point_count() const
        |             ^
        |             [[nodiscard]] 

• src/geode/geometry/nn_search.cpp:212:13: warning: [modernize-use-nodiscard]

  function 'kdtree_get_pt' should be marked [[nodiscard]]

    212 |             double kdtree_get_pt( size_t idx, size_t dim ) const
        |             ^
        |             [[nodiscard]] 

Have any feedback or feature suggestions? Share it here.

[BotellaA]

🎉 This PR is included in version 18.1.4-rc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[BotellaA]

🎉 This PR is included in version 18.1.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀