Basemap uses deprecated methods

[perimosocordiae]

I'm not sure who maintains basemap, or where the development takes place, but hopefully this gets routed to someone who knows.

With the current master branch of matplotlib and a fresh install of basemap-1.0.7, I trigger the following deprecation warnings:

from mpl_toolkits.basemap import Basemap
m = Basemap()

m.fillcontinents()
#/usr/local/lib/python2.7/dist-packages/mpl_toolkits/basemap/__init__.py:1767: MatplotlibDeprecationWarning: The get_axis_bgcolor function was deprecated in version 2.0. Use get_facecolor instead.
#  axisbgc = ax.get_axis_bgcolor()

m.scatter([1,2], [3,4])
#/usr/local/lib/python2.7/dist-packages/mpl_toolkits/basemap/__init__.py:3222: MatplotlibDeprecationWarning: The ishold function was deprecated in version 2.0.
#  b = ax.ishold()
#/usr/local/lib/python2.7/dist-packages/mpl_toolkits/basemap/__init__.py:3231: MatplotlibDeprecationWarning: axes.hold is deprecated.
#    See the API Changes document (http://matplotlib.org/api/api_changes.html)
#    for more details.
#  ax.hold(b)

[efiring]

hold has been taken care of, but not get_axis_bgcolor, so thanks for the report. I will open an issue on the basemap tracker, so this one can be closed.

[efiring]

Correction: get_axis_bgcolor is also being handled in basemap master, so I think all we need is a release to go with mpl v2.0.

[WeatherGod]

correct. that is the plan.

…

On Thu, Jan 12, 2017 at 4:49 PM, Eric Firing ***@***.***> wrote: Correction: get_axis_bgcolor is also being handled in basemap master, so I think all we need is a release to go with mpl v2.0. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#7808 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARy-Bim15vetR4APzT_v46Eft799imjks5rRp_igaJpZM4LiDoB> .

[roxyboy]

Has this fix already been released?

[WeatherGod]

Yes, the v1.1.0 release that is available on conda-forge. I have not bothered with releasing basemap to pypi, which is why it still has v1.0.7.

…

On Tue, Sep 26, 2017 at 3:01 PM, Takaya Uchida ***@***.***> wrote: Has this fix already been released? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7808 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARy-IovolQKqeIYHZnwH32KjVejeW9Eks5smUoPgaJpZM4LiDoB> .

[manojkken]

Any idea when this can be released?

[naught101]

You can use it now with conda install -c conda-forge basemap.

Basemap bugtracker is at https://github.com/matplotlib/basemap/issues

[gpetty]

Almost a year later, 'conda install basemap' still installs v1.0.7, which still throws an error when it tries to access ax.get_axis_bgcolor in matplotlib. Why hasn't v1.1.0 propagated to Anaconda yet?

[WeatherGod]

I don't control what anaconda puts out. I tagged v1.1.0 over a year ago, I don't know who to tell over at anaconda to update their packages. In the meantime, the recommended way to install basemap is through conda-forge.

…

On Tue, Sep 25, 2018 at 8:31 PM Grant Petty ***@***.***> wrote: Almost a year later, 'conda install basemap' still installs v1.0.7, which still throws an error when it tries to access ax.get_axis_bgcolor in matplotlib. Why hasn't v1.10 propagated to Anaconda yet? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7808 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARy-H7EFOHrHlCDzQbB-XY33ZS2BX82ks5uesr8gaJpZM4LiDoB> .

[efiring]

Ben, the problem is that even v1.1.0 fails on mpl 3.0, so a new release is needed. conda-forge will automatically update their package when a release is tagged. (ref. matplotlib/basemap#424)

[WeatherGod]

Ok, I just quickly pulled together a commit and tagged v1.2.0rel off of master. Let's see what happens.

…

On Tue, Sep 25, 2018 at 8:57 PM Eric Firing ***@***.***> wrote: Ben, the problem is that even v1.1.0 fails on mpl 3.0, so a new release is needed. conda-forge will automatically update their package when a release is tagged. (ref. #424 <#424>) — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#7808 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARy-MHhebQsQvavnVq-qdnaHTyEj11Zks5uetEKgaJpZM4LiDoB> .