Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: nearest_n_within does not limit num of items when not sorted #175

Merged
merged 4 commits into from
Jun 11, 2024

Conversation

sdd
Copy link
Owner

@sdd sdd commented Jun 11, 2024

Issue: #168

Copy link

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.42%. Comparing base (9651212) to head (936ed24).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #175      +/-   ##
==========================================
+ Coverage   95.30%   95.42%   +0.11%     
==========================================
  Files          52       52              
  Lines        7583     7650      +67     
  Branches     7583     7650      +67     
==========================================
+ Hits         7227     7300      +73     
+ Misses        335      329       -6     
  Partials       21       21              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sdd sdd force-pushed the fix-nearest-n-unsorted-bug branch from 1399d6e to a1cc36f Compare June 11, 2024 18:56
@sdd sdd force-pushed the fix-nearest-n-unsorted-bug branch from a1cc36f to 936ed24 Compare June 11, 2024 19:11
@sdd sdd merged commit 4c43660 into master Jun 11, 2024
11 checks passed
@sdd sdd deleted the fix-nearest-n-unsorted-bug branch June 11, 2024 19:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant