Skip to content

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