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

Fix issue with loading Bing map tiles #599

Merged
merged 2 commits into from
Dec 20, 2018

Conversation

pjreed
Copy link
Contributor

@pjreed pjreed commented Dec 18, 2018

The Bing API requires making an initial authorization call before it can
generate URLs for requesting tiles. A side effect of this was that if
the tile_map plugin tried to request any times before that initial call
completed, those tiles would fail to load, and they would never load
properly after that.

This adds a mechanism for the tile source to indicate whether it is
ready to load tiles or not, and the tile_map plugin will wait until it
is ready before requesting images, so Bing maps should always load
properly on startup.

The Bing API requires making an initial authorization call before it can
generate URLs for requesting tiles.  A side effect of this was that if
the tile_map plugin tried to request any times before that initial call
completed, those tiles would fail to load, and they would *never* load
properly after that.

This adds a mechanism for the tile source to indicate whether it is
ready to load tiles or not, and the tile_map plugin will wait until it
is ready before requesting images, so Bing maps should always load
properly on startup.
@pjreed pjreed requested a review from malban December 18, 2018 20:18
@pjreed pjreed merged commit 99ccb7a into swri-robotics:master Dec 20, 2018
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.

None yet

1 participant