-
Notifications
You must be signed in to change notification settings - Fork 46
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
[BUG] Maps miscalculates WMS BBOX #600
Comments
@opensearch-project/admin can you move this to the dashboard maps plugin repo? |
It seems like the bbox is always calculated for EPSG:3857. When I test with a local geoserver, I have to explicitly specify EPSG:3857 as the crs. Additionally the query url has EDIT:
and the bbox is hardcoded to use EPSG:3857. Therefore, I'm not quite sure why "WMS CRS" is even an input in the ui. Perhaps, it is best to lock the custom wms into using EPSG:3857. |
Describe the bug
After loading a WMS layer in Maps, the BBOX parameter appears to be off by five decimal places and is 100,000 larger than it should be. This effectively makes custom WMS layers unusable in Maps.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The bbox values should fall within the EPSG:4326/WGS 84 bounds of (longitude, latitude) -180, -90 and 180, 90. The result would be that WMS tiles should appear on the map according to what's in view at the time.
OpenSearch Version
2.12.0
Dashboards Version
2.12.0
Plugins
None. The Security plugin is also disabled.
Screenshots
None
Host/Environment (please complete the following information):
Additional context
Using the URL provided above, if I divide the BBOX values by 100,000, it produces a usable image.
https://mesonet.agron.iastate.edu/cgi-bin/wms/us/mrms_nn.cgi?service=WMS&version=1.1.1&request=GetMap&format=image/png&transparent=true&layers=mrms_p72h&styles=&srs=EPSG:4326&width=256&height=256&bbox=-106.44926307106785,50.09377085697312,-103.31840239250705,53.22463153553393
I also tested with WMS 1.3.0 and it appears to use the same, incorrect calculations.
The text was updated successfully, but these errors were encountered: