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

Documentation - Basemap #4706

Closed
chiragnagpal opened this issue Jul 15, 2015 · 6 comments
Closed

Documentation - Basemap #4706

chiragnagpal opened this issue Jul 15, 2015 · 6 comments
Assignees
Milestone

Comments

@chiragnagpal
Copy link

http://matplotlib.org/users/screenshots.html

Basemap demo say, "could not load basemap" Probably sphinx compiled on a machine without Basemap?

@tacaswell tacaswell added this to the next point release milestone Jul 15, 2015
@tacaswell
Copy link
Member

yup, that would be my fault, sorry about that!

@jenshnielsen
Copy link
Member

This happens easily because https://github.com/matplotlib/matplotlib/blob/master/doc/pyplots/plotmap.py just fails silently when basemap is not installed. I vote for either removing the silent failure, replac that example with a static image or removing it al together. We are already linking to basemap from http://matplotlib.org/mpl_toolkits/index.html with a static image

@WeatherGod
Copy link
Member

my vote is for a static image.

On Wed, Jul 15, 2015 at 7:12 PM, Jens Hedegaard Nielsen <
notifications@github.com> wrote:

This happens easily because
https://github.com/matplotlib/matplotlib/blob/master/doc/pyplots/plotmap.py
just fails silently when basemap is not installed. I vote for either
removing the silent failure, replac that example with a static image or
removing it al together. We are already linking to basemap from
http://matplotlib.org/mpl_toolkits/index.html with a static image


Reply to this email directly or view it on GitHub
#4706 (comment)
.

@tacaswell
Copy link
Member

👍 on static image

@mdboom
Copy link
Member

mdboom commented Jul 16, 2015

Yes. Disk space is much cheaper than mental bandwidth!

@tacaswell tacaswell modified the milestones: Color overhaul, next point release Aug 14, 2015
@mdboom mdboom modified the milestones: Color overhaul, next major release (2.0) Oct 8, 2015
@jenshnielsen jenshnielsen self-assigned this Mar 28, 2016
@jenshnielsen
Copy link
Member

Fixed by #6243

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants