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

Changed get_map zoom calculation to get min zoom level #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

metanoid
Copy link

@metanoid metanoid commented Jan 8, 2017

Sometimes, if the bounding box is either very wide or very tall, the zoom level chosen in get_map is too high (i.e. too zoomed in). I think the right behaviour is to choose the zoom level that captures the entire bounding box.

@dkahle
Copy link
Owner

dkahle commented Jan 9, 2017

Thanks, could you make a change in the NEWS file as well, please?

@espinielli
Copy link

Thanks for this. I am using the fix in a local copy of the function.
Any chance to have it in next CRAN version? (If I can ask, when? 😄 )

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