-
Notifications
You must be signed in to change notification settings - Fork 34
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
Initial load performance a bit slow #436
Comments
Tested today. Same old problem, too many default facets. e.g. in the test, all records search, 19s for the default 12 facets, 2.5s for the minimum of 1. Not a lot of ideas. Limiting to 4 facets is probably too few for some users. The default 12 I saw today is far too many.
Other opinions required. |
Here is a summary of manual tests than can be used for finding possible errors. There were some minor value formatting fixes that were also required. The code is still doing 2 requests, instead of 1, when data quality filters are enabled because I did not think it looked good when the displayed record count contained a spinner for the "of" count, e.g. If all facet groups are open, and both data quality sections are open, I expect the larger number of individual facet requests will have a slightly larger load on the backend. However facet groups are not open by default so this will not be the typical use case. When comparing the performance, using the default facets and all records, the initial page response time did report ~2s vs 11s. This is not consistent and there are other factors that apply. Test removal of empty facet groups:
Test on-demand retrieval/display of selected Data Profile
Test "Narrow your results" group opening/closing. Excludes Data Profile group.
Test all facets (not using :, but probably should be)
|
…performance #436 get facet information for list view only when required
The initial page load performance of
biocache-hubs
is slow. For me there is a 6-8 secs lag at the start of the load.GBIF report this is more like 11 seconds for them from Copenhagen.
The text was updated successfully, but these errors were encountered: