-
Notifications
You must be signed in to change notification settings - Fork 3
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
Set the whole known service after appending the new batch #9
Labels
bug
Something isn't working
Comments
Not related to the current deployment issue. |
@Superskyyy We have fixed most od known issue, but can't tell how to do this properly. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To prevent edge cases.
A service should at least seen 20 unique uris before executing clustering. Current line 89 in server.py is not a correct mechanism to enforce that. So some service will unnecessarily trigger clustering after 20* interval.
Change the counter to a set. Append new and return empty until reach 20.
The text was updated successfully, but these errors were encountered: