-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ENH] simplify logic for when to persist index changes (re-apply with fix) #2545
base: main
Are you sure you want to change the base?
Conversation
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
399c0a8
to
807c4c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be a better name for this
12a0112
to
4afc352
Compare
@@ -276,7 +276,7 @@ def ann_accuracy( | |||
|
|||
try: | |||
note( | |||
f"recall: {recall}, missing {missing} out of {size}, accuracy threshold {accuracy_threshold}" | |||
f"# recall: {recall}, missing {missing} out of {size}, accuracy threshold {accuracy_threshold}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes it slightly easier to copy/paste failing state machine repro
aed02f9
to
4d920d6
Compare
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @codetheweb and the rest of your teammates on |
4d920d6
to
d84489d
Compare
d84489d
to
7c1d6d0
Compare
History:
Should not be merged until hnswlib version is bumped (bug around persisting a single item was recently fixed).