Skip to content

Commit

Permalink
Feature/improve popup trigger performance (hpi-sam#586)
Browse files Browse the repository at this point in the history
* Updates all packages
    * Only version bumps on minor and patch level
    * OpenLayers now sets willReadFrequently on the canvas rendering contexts (https://github.com/openlayers/openlayers/releases/tag/v7.2.0). A quick test showed this improves the performance from ca. 150ms per click to ca. 120ms. There was no difference when the map was not moved in between clicks.
* Improvements in hasFeatureAtPixel in PopupManager
    * Early return in hasFeatureAtPixel
    * Skip not relevant layers
    * This improved the performance from ca. 120 ms to ca. 35ms
  • Loading branch information
Dassderdie authored Dec 22, 2022
1 parent 72a544d commit 29d5e5b
Show file tree
Hide file tree
Showing 13 changed files with 619 additions and 188 deletions.
112 changes: 56 additions & 56 deletions backend/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions backend/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,9 @@
"@types/lodash-es": "^4.17.6",
"@types/node": "^16",
"@types/supertest": "^2.0.12",
"@typescript-eslint/eslint-plugin": "5.46.1",
"@typescript-eslint/parser": "5.46.1",
"eslint": "^8.29.0",
"@typescript-eslint/eslint-plugin": "5.47.0",
"@typescript-eslint/parser": "5.47.0",
"eslint": "^8.30.0",
"eslint-config-prettier": "^8.5.0",
"eslint-plugin-import": "~2.26.0",
"eslint-plugin-unicorn": "^45.0.2",
Expand Down
Loading

0 comments on commit 29d5e5b

Please sign in to comment.