-
Notifications
You must be signed in to change notification settings - Fork 989
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
Leverage WebWorker for clustering #1089
Comments
Hi @Etienne-M this indeed sounds like a significant amount of refactoring as previously mentioned. Perhaps we can use this issue as a poll for such improvement. If we got a sufficient amount of attention we can def prioritize it. In the meantime, feel free to open a PR if you had any solution as well we can review it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Clustering a large amount of probes can take ~1 seconds by itself. All of this time is currently spent on the main thread, which yields a poor user experience.
We could move this computationally intensive logic inside a web worker to free up the main thread.
This issue has already been opened in the past, but it was previously closed: #131
The text was updated successfully, but these errors were encountered: