Skip to content
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

perf(Stats): use Web Workers #21

Open
maxpatiiuk opened this issue Feb 11, 2024 · 0 comments
Open

perf(Stats): use Web Workers #21

maxpatiiuk opened this issue Feb 11, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@maxpatiiuk
Copy link
Owner

Stats generation, when run on my text-hoarder-store, takes 7 min with --precise-stats and 40s without --precise-stats
By using web workers, we can make this a lot faster, though that would add complexity, and make code reuse between stats and pre-processor harder.
Plus, I am not running stats generation often, so performance is not important.
And --precise-stats aren't too much more precise than the default, at a large performance penalty

Thus, it might be best to consider using web workers only in the future, once pre-processor code is finished, and once there are more users of the Cli

@maxpatiiuk maxpatiiuk added the enhancement New feature or request label Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🆕 New
Development

No branches or pull requests

1 participant