Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Raster tile overlay with TMS coordinates (non-flipped Y) #2270

Merged
merged 1 commit into from
Aug 16, 2016

Conversation

kkaefer
Copy link
Contributor

@kkaefer kkaefer commented Aug 15, 2016

Hi,
We want to provide our own raster overlay for a certain area but MapBox flips the y coordinate while our tile service does not. We are providing a style.json and a tileset.json file like this:

style.json

    {
  "version": 7,
  "sources": {
    "simple-tiles": {
      "type": "raster",
      "url": "http://localhost:9000/tileset.json",
      "tileSize": 256
    }
  },
  "layers": [
    {
      "id": "background",
      "type": "background",
      "paint": {
        "background-color": "rgb(140,215,225)"
      }
    },
    {
      "id": "simple-tiles",
      "type": "raster",
      "source": "simple-tiles"
    }
  ]
}

tileset.json

{
  "tiles": [
    "http://localhost:9000/proxy.php?z={z}&x={x}&y={y}"
  ],
  "tilejson": "2.1.0",
  "scheme": "tms"
}

The proxy-php currently only flips the y-coordinate back, we want to get rid of the proxy.php file. How can we achieve non-flipped-y tile loading?

@drmarkpowell
Copy link

+1 for this feature. We're going to need to re-tile all our Mars maps to match the other standard and maintain both versions to support legacy tools and Mapbox map views...would be better on our end to have only one tileset.

@mention-bot
Copy link

@kkaefer, thanks for your PR! By analyzing this pull request, we identified @jfirebaugh, @1ec5 and @mourner to be potential reviewers.

@1ec5
Copy link
Contributor

1ec5 commented Aug 15, 2016

👍 Would you do the honors and add a blurb to the iOS and macOS changelogs?

@kkaefer
Copy link
Contributor

kkaefer commented Aug 15, 2016

JS-side PR was already merged: mapbox/mapbox-gl-js#2565

@kkaefer kkaefer force-pushed the 2270-tilejson-tms-scheme branch 4 times, most recently from 9c9d803 to eb96328 Compare August 16, 2016 08:26
@kkaefer kkaefer merged commit 1817660 into master Aug 16, 2016
@kkaefer kkaefer deleted the 2270-tilejson-tms-scheme branch August 16, 2016 09:05
ansis added a commit that referenced this pull request Dec 15, 2016
ported from -js: 0b5520fa5ab2a4659d80dcffa8b035a0d84fe1ca

This should fix the issue behind #2270 and remove the need for the hack
added in #3740.
ansis added a commit that referenced this pull request Dec 21, 2016
ported from -js: 0b5520fa5ab2a4659d80dcffa8b035a0d84fe1ca

This should fix the issue behind #2270 and remove the need for the hack
added in #3740.
jfirebaugh pushed a commit that referenced this pull request Dec 21, 2016
ported from -js: 0b5520fa5ab2a4659d80dcffa8b035a0d84fe1ca

This should fix the issue behind #2270 and remove the need for the hack
added in #3740.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants