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

Loading spatial classification takes too long #472

Closed
acka47 opened this issue Apr 1, 2019 · 5 comments · Fixed by #473
Closed

Loading spatial classification takes too long #472

acka47 opened this issue Apr 1, 2019 · 5 comments · Fixed by #473
Assignees

Comments

@acka47
Copy link
Contributor

acka47 commented Apr 1, 2019

Loading https://nwbib.de/spatial in the browser (Chromium) takes more than ten seconds. We noticed this ourselves and H.F. also mentioned it on the mailing list on 2019-03-22:

Wenn man die Regionensystematik ansteuert, dann fallen uns zumeist längere Ladezeiten auf als bei der Sachsystematik. Hängt das mit den vielen Ortsdaten zusammen? Und könnte man dagegen etwas tun?

Testing it online with https://www.webpagetest.org, the first view takes 25s, see https://www.webpagetest.org/result/190401_7J_d095ed9fc7de3bd68d061f450faf0215/. It looks like caching isn't enabled. With https://www.giftofspeed.com/ it takes 12s and the page also gives back some more hints on what to improve.

@fsteeg
Copy link
Member

fsteeg commented Apr 5, 2019

Deployed to test: https://test.nwbib.de/spatial

(Caching was working, was a client side Javascript issue)

@fsteeg fsteeg assigned acka47 and unassigned fsteeg Apr 5, 2019
@fsteeg fsteeg removed the in progress label Apr 5, 2019
@acka47
Copy link
Contributor Author

acka47 commented Apr 8, 2019

This is great. Much faster now. +1

@acka47 acka47 removed their assignment Apr 8, 2019
@acka47 acka47 added this to Review in lobid board Apr 8, 2019
lobid board automation moved this from Review to Done Apr 8, 2019
@acka47
Copy link
Contributor Author

acka47 commented Apr 26, 2019

Wie H.F. mitgeteilt hat, gibt es immer noch Probleme mit Firefox, die ich gerade an meinem Telearbeitsrechner reproduzieren konnte (Ladezeit war bei mir nahe 20 s).

On 23.04.19 17:54, H.F. wrote:

wir haben unsere Erfahrungen aus dem NWBib-Kollegenkreis Düsseldorf und Münster einmal zusammengetragen. Sie ergeben immerhin ein Bild. Dabei wurde mit Mozilla Firefox getestet. Generell fiel auf, dass beim ersten Laden der Raumsystematik (Regionen) nach Aufruf des Browsers nach wie vor mehr Zeit benötigt wird, zwischen 20 und 28 Sekunden. Bei Folgeaufrufen geht das dann sehr viel schneller, ca. 3 sec.

Ein paar Aufrufe vom Privat-PC zu Hause kamen zum gleichen Ergebnis.

Wenn ich mit dem Internetexplorer teste, dann geht das von Anfang an sehr rasch.

@acka47 acka47 reopened this Apr 26, 2019
@acka47 acka47 removed this from Done in lobid board Apr 26, 2019
@acka47 acka47 added this to Backlog in NWBib-Raumsystematik via automation Apr 26, 2019
@acka47 acka47 added this to Backlog in lobid board via automation Apr 26, 2019
@acka47 acka47 removed this from Backlog in NWBib-Raumsystematik Apr 26, 2019
@acka47 acka47 moved this from Backlog to Ready in lobid board Apr 26, 2019
@acka47 acka47 removed the review label Apr 26, 2019
fsteeg added a commit that referenced this issue Apr 26, 2019
@fsteeg
Copy link
Member

fsteeg commented Apr 26, 2019

Ich kann das nicht reproduzieren. Habe den Cache für die Klassifikationen auf dauerhaft gestellt, vorher war das 1 Tag. Ich könnte mir vorstellen dass ihr den Fall hattet das das für den Tag ausgelaufen war und deshalb lange dauerte. Auf test deployed:

https://test.nwbib.de/spatial

@fsteeg fsteeg added this to Backlog in NWBib-Raumsystematik via automation Apr 26, 2019
@fsteeg fsteeg removed this from Ready in lobid board Apr 26, 2019
@fsteeg fsteeg moved this from Backlog to Review in NWBib-Raumsystematik Apr 26, 2019
@acka47
Copy link
Contributor Author

acka47 commented Apr 26, 2019

+1 Hab es gerade an einem anderen Rechner ausprobiert, da lief es sowohl auf test als auch auf production gut...

@acka47 acka47 assigned fsteeg and unassigned acka47 Apr 26, 2019
@acka47 acka47 moved this from Review to Deploy in NWBib-Raumsystematik Apr 26, 2019
@fsteeg fsteeg closed this as completed in ff144f8 Apr 26, 2019
NWBib-Raumsystematik automation moved this from Deploy to Done Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants