Page MenuHomePhabricator

Investigation: Explore ways to implement nearby auto-refresh to avoid heavy load
Closed, ResolvedPublic5 Estimated Story Points

Description

Look into ways to make auto-refresh feasible performance wise. Can be done by implementing PoC testing different approaches and parameters.

Ideas:

  • Debounce based on coordinate and zoom.
  • Heavily round the coordinates, so they are only different when the map is moved a lot.
  • We might want to play with the radius calculated to fit the bounding box. If it's a bit bigger we might not need refresh if the box moves just a little.

Event Timeline

WMDE-Fisch set the point value for this task to 5.May 18 2022, 8:51 AM
Aklapper renamed this task from Investigaton: Explore ways to implement nearby auto-refresh to avoid heavy load to Investigation: Explore ways to implement nearby auto-refresh to avoid heavy load.Aug 8 2022, 10:31 AM

Change 821723 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/extensions/Kartographer@master] Include zoom factor into de-bouncing calculation

https://gerrit.wikimedia.org/r/821723

Change 822065 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] [POC] Visualize the nearby query range

https://gerrit.wikimedia.org/r/822065

Change 821723 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] Include zoom factor into de-bouncing calculation

https://gerrit.wikimedia.org/r/821723

Change 823164 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] Simple auto-refresh for nearby pages feature

https://gerrit.wikimedia.org/r/823164

Change 823166 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] Merge all incoming nearby data

https://gerrit.wikimedia.org/r/823166

Change 823174 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] Don't add duplicate points to the nearby layer

https://gerrit.wikimedia.org/r/823174

Change 823179 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] [POC] Use more memory efficient hash function

https://gerrit.wikimedia.org/r/823179

Change 823591 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/Kartographer@master] Add pin icon to "show nearby articles" button

https://gerrit.wikimedia.org/r/823591

One thing we could start experimenting with in this context would be storing the areas from previous requests. The library offers checks for overlapping areas, so we could check if a new request would overlap with what we already have and then don't do a request.

Change 823591 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] Add pin icon to "show nearby articles" button

https://gerrit.wikimedia.org/r/823591

Change 823164 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] Simple auto-refresh for nearby pages feature

https://gerrit.wikimedia.org/r/823164

Change 822065 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/extensions/Kartographer@master] [POC] Visualize the nearby query range

Reason:

https://gerrit.wikimedia.org/r/822065