-
Notifications
You must be signed in to change notification settings - Fork 444
Groupe open/panoramax streetview support #11822
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
base: master
Are you sure you want to change the base?
Groupe open/panoramax streetview support #11822
Conversation
|
@nrc97 thank you so much for contributing with this. We will review as soon as possible. |
map.forEachFeatureAtPixel in openlayers/Map.jsx adapted for detection of features of type RenderFeature used for MVT layers. By doing that, MVT layers are clickable and the intersected features are return on clic TileProviderLayer for being able to create MVT layer and display it.
341255c to
5eed6a3
Compare
- Create a viewer using pnx-photo-viewer to display pictures. - Add a Panoramax VectorTileLayer using OpenLayers styling to match other providers. - Handle click events: retrieve picture data directly from the layer datas, or call the API if the data is not found.
…h, Italian and German
5eed6a3 to
6e7f4bf
Compare
|
Hi @nrc97 if you are still working on this, I kindly ask you to convert this PR to a Draft PR to avoid reviewing if you haven't completed yest the development. Thank you so much again for the collaboration and the contribution. |
|
Is this a tool dedicated to OpenLayers ? (Mapstore offering several map libraries) |
|
Fwiw, i'm not in a position to judge the code itself, but from basic testing against a locally deployed panoramax instance and a locally running mapstore from this PR, this works fine, eg sequences are loaded as MVT from the api, and clicking on the sequence brings up a window showing the picture. Features are basic (eg no link to the complete sequence, which can be useful, nor filters like on panoramax web viewer), but there's always the link to the original picture in panoramax, so that's fine. if i had a user-level remark to do, when the streetview window has its default scale (eg 'smallish'), the 'back' arrow is a bit hidden by the licence footer for the image. one has to make the window a bit higher to easily interact with the arrow. |
@nrc97 can we finally schedule a review session for this? Thank you for confirming that. |
Description
This PR introduces support for Panoramax as an additional Street View provider in MapStore2. Users can now visualize street‑level imagery using Panoramax alongside existing providers. The feature includes configuration options, i18n strings, attribution handling, and provider selection at runtime.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
Panoramax is not supportted as streetview provider
What is the new behavior?

We can now use panoramax as provider for the streetview plugin.
Breaking change
Other useful information
Panoramax OpenAPI Documentation: https://docs.panoramax.fr/api/api/openapi/
Panoramax Viewer Documentation: https://docs.panoramax.fr/web-viewer/#basic-usage