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

Slow loading large locations lists #1

Open
pettazz opened this issue Mar 8, 2021 · 1 comment
Open

Slow loading large locations lists #1

pettazz opened this issue Mar 8, 2021 · 1 comment

Comments

@pettazz
Copy link
Member

pettazz commented Mar 8, 2021

Now that the location list is pretty large, the fact that we do 3 separate full loops (map(), filter(), sort()) to merge, then filter, then sort is noticeably slow, so they should be combined to reduce as much as possible.

@pettazz
Copy link
Member Author

pettazz commented Mar 8, 2021

nvm, looks like the filtering and sorting is milliseconds quick, it's alpines generating the template and adding it to the DOM that takes a few seconds. Not sure there's anything we can do to improve this, but there may be performance tweaks to help alpine.

@pettazz pettazz changed the title Combine merge/filter/sort Slow loading large locations lists Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant