TUM-Dev / NavigaTUM

Navigating around TUM with excellence – A website and API to search for rooms, buildings and other places
https://nav.tum.de
GNU General Public License v3.0
38 stars 8 forks source link

On slow networks, deduplication drops requests #1274

Open TCA-Bot opened 6 days ago

TCA-Bot commented 6 days ago

"Institute for Advanced Study (IAS)" wird bei der Suche nach "IAS" nicht in den ersten Suchergebnissen angezeigt, obwohl die Abkürzung Teil des Gebäudenamens auf nav.tum.de ist.

CommanderStorm commented 6 days ago

Cannot reproduce

Screenshot_20240702_143026_Chrome.jpg

AntonMC-Github commented 6 days ago

Wird bei mir jetzt auch angezeigt, keine Ahnung, was da eben nicht ging - vielleicht hat die Suche sich bei meinem langsamen Internet nur nicht aktualisiert, issue kann also gerne weg :)

CommanderStorm commented 6 days ago

Das issue ist schon valide.

Eventuell muss ich mir mal diese settings genauer anschauen wie sie sich auf connected mobility auswirken.

(momentan: dedupe=defer)

dedupe: avoid fetching same key more than once at a time. Possible options:

  • cancel - cancels existing requests when a new one is made
  • defer - does not make new requests at all if there is a pending request

Eventuell sollte man nach einer gewissen zeit (200-300ms, sonnst zu disruptiv) auch statt alten suchergebnissen einen spinner anzeigen, falls wir den request neu versuchen müssen. Warscheinlich kann ich den Wert hier auch etwas smarter setzen (das ist momentan bei 5s um daten beim nutzer zu sparen => falls nutzer gutes internet hat sollten wir alle 500ms retryen)

AntonMC-Github commented 6 days ago
Hab's nochmal repliziert bekommen - diesmal sogar mit Screenshot :)

![](https://github.com/TUM-Dev/NavigaTUM/assets/35605744/20dff904-696e-4289-b63c-0ea9fc602de7)

Umgebung ist folgende: WLAN aus Italien (Ping nach Nürnberg: 29ms) mit 700KBit Down / 30 MBit Up

Ich hab' direkt nach Laden der Website die Suche verwendet.

Wenn ich dann nochmal 10-20 Sekunden gewartet habe und dann nochmal das ganze eingegeben hab' ging's dann

![](https://github.com/TUM-Dev/NavigaTUM/assets/35605744/1f23ca1e-7ad1-4fcc-b429-4b307ae085cd)

Die Website wurde auch bei der ersten Eingabe schon als fertig geladen angezeigt

CommanderStorm commented 6 days ago

Super weird. Can reproduce this by doing (1-3)

image

CommanderStorm commented 5 days ago

FYI: als hotfix habe ich dedupe=cancel gesetzt. Das ist aber nicht optimal, da damit erst alle inflight requests sofort gecancelt werden (anstatt schön zu pipelinen)