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.

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

Support for EPSG:3765 (regional alias for EPSG:4326) #832

Closed
criskurtin opened this issue Feb 4, 2025 · 1 comment
Closed

Support for EPSG:3765 (regional alias for EPSG:4326) #832

criskurtin opened this issue Feb 4, 2025 · 1 comment

Comments

@criskurtin
Copy link

Hi all!

I want to use Go Map!! for some local mapping while I'm outside with my iPhone or iPad. To do that, I'd like to have access to some WMS layers of the Croatian spatial development and general urban plans. Unfortunately, Croatian governmental WMS servers often advertise only EPSG:3765 and CRS:84, as is the case here: https://geoportal.zagreb.hr/Public/GUPZagreb_Public/MapServer/WMSServer?request=getcapabilities

EPSG:4326 is even advertised in the base layer, as it should be, but it's omitted from the nested layers.
3765 is just a regional realisation of 4326 so no reprojection is required between these. I wanted to add the support by myself (I still can), but I'd like to consult here first. I see there are two conditional statements in the code for 4326 at the moment, one for URL generation and one for converting tile to WMS coordinates. If 3765 is added, these statements would need to have 3765 added to them too. This could become messy if additional 4326 aliases would be added later on, so perhaps it would be good to separate these aliases into some list and replace the conditionals with something like if epsg4326Aliases.contains(projection).

I'd gladly make the service providers correct their WMS layer definitions, but that's probably not going to happen any time soon.

@bryceco
Copy link
Owner

bryceco commented Feb 4, 2025

if epsg4326Aliases.contains(projection)

I like this approach. Thanks for looking into this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants