Skip to content

Making GetTileCoordFor methods public avaible #2973

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

Merged
merged 2 commits into from
Dec 1, 2014

Conversation

pksorensen
Copy link
Contributor

I am using tilegrid to request elevation data tiles side by side of a raster layer. It would be nice if these two method was public avaible such I could use the tilegrid to convert mouse coordinates to tile coordinates to request backend tiles.

The elevation tiles are not shown as a layer directly but rather used to compute and show information in relationship to mouse curser or map view.

This would save me the time of adding another tilegrid implementation in my application specific context.

I am using tilegrid to request elevation data tiles side by side of a raster layer. It would be nice if these two method was public avaible such I could use the tilegrid to convert mouse coordinates to tile coordinates to request backend tiles. 

The elevation tiles are not shown as a layer directly but rather used to compute and show information in relationship to mouse curser or map view.

This would save me the time of adding another tilegrid implementation in my application specific context.
@elemoine
Copy link
Member

+1

We can use @api rather than @api stable if we don't want to a commit to a tile grid API at this point.

@tschaub, any opinion about this?

@pksorensen
Copy link
Contributor Author

I just enabled them in my own private build for now. Just wanted to let you know that its working out good. I ported your tilegrid api for internal backend stuff also (c#) https://github.com/s-innovations/gis/tree/master/SInnovations.Gis.TileGrid.

We also uses mapproxy in our setup and so far the two grid implementations have great overlap and its been easy for a none gis developer to understand all of it so a big thumbs up from me on the work you guys do.

@elemoine
Copy link
Member

Please remove the stable marker and I'll merge this. Thanks.

@tschaub
Copy link
Member

tschaub commented Dec 1, 2014

I agree with making them exportable, but not marked as stable.

elemoine pushed a commit that referenced this pull request Dec 1, 2014
Making GetTileCoordFor methods public avaible
@elemoine elemoine merged commit 4734530 into openlayers:master Dec 1, 2014
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

Successfully merging this pull request may close these issues.

3 participants