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.

Sign up for GitHub

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

Jump to bottom

[POC] MapLibre layer #981

Draft
wants to merge 9 commits into
base: develop
Choose a base branch
from
Draft

[POC] MapLibre layer #981

wants to merge 9 commits into from

Conversation

pakb
Copy link
Contributor

@pakb pakb commented Jul 4, 2024 •
edited
Loading

@github-actions github-actions bot changed the title [POC] MapLibre layer [POC] MapLibre layer - #minor Jul 4, 2024
@cypress Cypress
Copy link

cypress bot commented Jul 4, 2024 •
edited
Loading

web-mapviewer    Run #4624

Run Properties:  status check failed Failed #4624  •  git commit 75c354fbc5: WIP Print view
Project web-mapviewer
Branch Review poc-maplibre-layer
Run status status check failed Failed #4624
Run duration 33m 36s
Commit git commit 75c354fbc5: WIP Print view
Committer Pascal Barth
View all properties for this run ↗︎
Test results
Tests that failed  Failures 149
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 20
Tests that did not run due to a failure in a mocha hook  Skipped 37
Tests that passed  Passing 0
View all changes introduced in this branch ↗︎

Tests for review

Failed  layers.cy.js • 30 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Test of layer handling > Layer in URL at app startup > starts without any visible layer added opening the app without layers URL param Screenshots
Test of layer handling > Layer in URL at app startup > adds a layers with config to the map when opening the app layers URL param Screenshots
Test of layer handling > Layer in URL at app startup > uses the default timestamp of a time enabled layer when not specified in the URL Screenshots
Test of layer handling > Layer in URL at app startup > sets the timestamp of a layer when specified in the layers URL param Screenshots
... > reads and adds an external WMS correctly Screenshots
... > reads and adds an external WMTS correctly Screenshots
... > reads and adds an external WMTS correctly but doesn't show the invisible layer Screenshots
... > handles errors correctly Screenshots
Test of layer handling > Background layer in URL at app startup > sets the background to the topic default if none is defined in the URL Screenshots
Test of layer handling > Background layer in URL at app startup > sets the background to the topic default if none is defined in the URL, even if a layer (out of topic scope) is defined in it Screenshots
The first 10 failed tests are shown, see all 30 tests in Cypress Cloud.
Failed  legacyParamImport.cy.js • 18 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Test on legacy param import > Coordinates import > transfers valid params to the hash part without changing them Test Replay Screenshots
Test on legacy param import > Coordinates import > loads correctly if params are already behind the hash Test Replay Screenshots
Test on legacy param import > Coordinates import > reproject LV95 coordinates param to EPSG:4326 Test Replay Screenshots
Test on legacy param import > Coordinates import > center where expected when given a X, Y coordinate in LV95 Test Replay Screenshots
Test on legacy param import > Coordinates import > center where expected when given a X, Y coordinate in LV03 Test Replay Screenshots
Test on legacy param import > Layers import > Combines all old layers_*** params into the new one Test Replay Screenshots
Test on legacy param import > Layers import > is able to import an external KML from a legacy param Test Replay Screenshots
Test on legacy param import > Layers import > is able to import an external KML from a legacy adminId query param Test Replay Screenshots
Test on legacy param import > Layers import > don't keep KML adminId in URL after import Test Replay Screenshots
Test on legacy param import > Layers import > is able to import an external KML from a legacy adminId query param with other layers Test Replay Screenshots
The first 10 failed tests are shown, see all 18 tests in Cypress Cloud.
Failed  drawing.cy.js • 11 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Drawing module tests > receives an empty KML and can use drawing mode Test Replay Screenshots
Drawing module tests > Drawing mode/tools > can create marker/icons and edit them Test Replay Screenshots
Drawing module tests > KML management > deletes the drawing when confirming the delete modal Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer in the layer list / URL params correctly Test Replay Screenshots
Drawing module tests > KML management > keeps the KML after a page reload, and creates a copy if it is then edited Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer correctly if it comes attached with an adminId at startup from a legacy URL Test Replay Screenshots
Drawing module tests > others > doesn't save an empty drawing (if not modified) Test Replay Screenshots
Drawing module tests > others > can export the drawing/profile in multiple formats Test Replay Screenshots
Drawing module tests > others > generates short links when sharing a drawing Test Replay Screenshots
Drawing module tests > others > shows a profile of a line/measure coming from service-alti data Test Replay Screenshots
The first 10 failed tests are shown, see all 11 tests in Cypress Cloud.
Failed  importToolFile.cy.js • 3 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
The Import File Tool > Import KML file Test Replay Screenshots
The Import File Tool > Import KML file error handling Test Replay Screenshots
The Import File Tool > Import GPX file Test Replay Screenshots
Failed  print.cy.js • 9 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Testing print > Print UI > should populate the UI with the capabilities from mapfishprint Test Replay Screenshots
Testing print > Send print request > should send a print request to mapfishprint (basic parameters) Test Replay Screenshots
Testing print > Send print request with layers > should send a print request to mapfishprint (with layers added) Test Replay Screenshots
Testing print > Send print request with layers > should send a print request correctly to mapfishprint (with KML layer) Test Replay Screenshots
Testing print > Send print request with layers > should send a print request correctly to mapfishprint with GPX layer Test Replay Screenshots
Testing print > Send print request with layers > should send a print request correctly to mapfishprint (icon and label) Test Replay Screenshots
Testing print > Send print request with layers > should send a print request correctly to mapfishprint (KML from old geoadmin) Test Replay Screenshots
Testing print > Send print request with external layers > prints external WMS correctly Test Replay Screenshots
Testing print > Send print request with external layers > prints external WMTS correctly Test Replay Screenshots

The first 5 failed specs are shown, see all 31 specs in Cypress Cloud.

@pakb pakb force-pushed the poc-maplibre-layer branch from 9d4476d to ea1d620 Compare August 26, 2024 11:09
@pakb pakb force-pushed the poc-maplibre-layer branch 2 times, most recently from 829e4e2 to 016cc8b Compare September 20, 2024 07:45
@pakb pakb changed the base branch from master to develop September 20, 2024 07:46
@pakb pakb changed the title [POC] MapLibre layer - #minor [POC] MapLibre layer Sep 20, 2024
@pakb pakb force-pushed the poc-maplibre-layer branch 3 times, most recently from 41e4b94 to 1925e79 Compare September 20, 2024 08:15
pakb added a commit to pakb/ol-maplibre-layer that referenced this pull request Jan 8, 2025
@pakb
Add support for non Mercator projection in OL view
e3a76de 
give the opportunity to translate the zoom level before giving it to MapLibre. This makes it possible to have OL functioning with a non-Mercator projection system, and still be able to have MapLibre receiving a valid Mercator zoom level to keep the maps in sync.

You can see this code in action here : geoadmin/web-mapviewer#981
where we have our OL instance working in LV95.
@pakb pakb mentioned this pull request Jan 8, 2025
Merged
@pakb pakb force-pushed the poc-maplibre-layer branch from 1925e79 to 479db78 Compare January 8, 2025 11:15
@pakb pakb force-pushed the poc-maplibre-layer branch 5 times, most recently from 4c32505 to d41c210 Compare January 21, 2025 12:41
@pakb pakb force-pushed the poc-maplibre-layer branch from 1bb0430 to 8559cd4 Compare February 18, 2025 12:52
pakb added 9 commits February 21, 2025 07:38
@pakb
[POC] MapLibre layer
71032b4 
replaces all BG layers with a VectorTile equivalent
switch to mercator as default projection to make it happen (not possible yet to mix projection systems, needs some work done on the geoblocks/ol-maplibre-layer library)

fix INT staging URL : no sys service serves theses styles yet
update to MapLibre 5.0.0
@pakb
[POC] Add events when MapLibre and OL finished loading
da1ab23 
so that our print vector tiles POC can hook up to them
@pakb
[POC] Support on-the-fly LV95 reprojection
ae2d5a2 
copying geoblocks/ol-maplibre-layer/ locally to be able to edit it and add the on the fly capabilities (will create a PR there when done)

Making our coordinate system calculate resolution without threshold (especially for LV95) so that it can then be used to calculate a mercator zoom level
@pakb
[POC] print vectortiles : print mode
98c8aa7 
adding a flag making OL requests next zoom level tiles when in between zoom levels
@pakb
[POC] print vectortiles : only one event triggered for all map renderers
fe760a7
@pakb
[POC] fix MapLibre ready event
67763a7
@pakb
Adding vue/vite dev tools
b962eb8
@pakb
Move debug toolbar to top App component
903ad25 
so that it will be added to embed and future print view without needing to change anything

also changing a bit the CSS of the views, so that it is not using an ID to style each view, but a class (doesn't really make sense to have the embed view have the ID map-view)
@pakb
WIP Print view
75c354f
@pakb pakb force-pushed the poc-maplibre-layer branch from d8ded79 to 75c354f Compare February 21, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pakb