-
Notifications
You must be signed in to change notification settings - Fork 397
Arcgisimage fails #242
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
Comments
Which version of matplotlib are you using? I am pretty sure this is fixed On Mon, Dec 14, 2015 at 1:02 PM, Bastien Jacot-Guillarmod <
|
I use matplotlib 1.5.0 (arch linux) |
Ah, I think I know what it is... do you have Pillow installed? On Mon, Dec 14, 2015 at 1:11 PM, Bastien Jacot-Guillarmod <
|
Yes, I do. Version 3.0.0 |
I uninstalled it and retry. The error remains though. |
Does example/testarcgis.py work for you? |
Here is what worked for me: https://github.com/guziy/PyNotebooks/blob/master/basemap_issue_%23242.ipynb Cheers 2015-12-14 13:30 GMT-05:00 Micah Cochran notifications@github.com:
Sasha |
testarcgist.py fails with the same error it seems.
|
Ah, I have been looking at this wrong... Just noticed that you are using @mdboom, could the recent error-checking changes we made to _png.h in On Mon, Dec 14, 2015 at 1:33 PM, Huziy Oleksandr (Sasha) <
|
I think plt.imread is a problem here, for some reason it does not accept Cheers 2015-12-14 13:40 GMT-05:00 Benjamin Root notifications@github.com:
Sasha |
btw, a possible fix in our code might be to do something similar to that in
On Mon, Dec 14, 2015 at 1:40 PM, Benjamin Root ben.v.root@gmail.com wrote:
|
@guzly, What has changed is that @mdboom (and I, to a much lesser extent) made some tiny fixes to the PNG reading code to add error checking in case that was the cause of some random, spurious issues with Travis in matplotlib. It is entirely possible that this error-checking is too tight and is triggering when it shouldn't be. |
2015-12-14 13:45 GMT-05:00 Benjamin Root notifications@github.com:
I am not sure if it won't cause memory problems, but from the other side I CheersSasha |
@guziy Thanks for the gist! I was able to make it work! |
This is potentially fixed by matplotlib/matplotlib#5910 |
Fixed in master. Seeing if it needs to be backported to 1.5.x. |
This has been backported to v1.5.x as well. |
I've updated my matplotlib from 1.5.0 to 1.5.1 today using pip (Python 3), and I still have this problem. Do I need to install matplotlib from github to get the fix? |
The fix will be in the upcoming v1.5.2, and in the upcoming v2.0 releases. On Sun, Feb 14, 2016 at 4:40 AM, einarbmag notifications@github.com wrote:
|
I tried a simple map and add some
arcgisimage
, but it failed.Here is the code:
And the error:
After debugging around, I found that
imread
does not likehttp.request.HttpResponse
, but works fine with file path.Regards,
Bastien
The text was updated successfully, but these errors were encountered: