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(web): optimize date groups #7593
Conversation
Deploying with Cloudflare Pages
|
Does this solve issue #5030 ? |
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.
Great stuff! :)
No, that one is because we only do a rough estimate of height initially and update as we fetch the assets. The justified layout makes it hard to get it exact. There's probably room for improvement there though. |
This reverts commit 762c468.
Description
The algorithm for grouping by date is inefficient and doesn't scale well. There's also no caching, so revisiting buckets re-groups again. This PR optimizes this with a more efficient algorithm and ensures that a given bucket will only ever be grouped into dates once.
How Has This Been Tested?
Tested with a console log during grouping to check that a given bucket is only grouped once, even after scrubbing away and back to the bucket.