lgienapp / aquarel

Styling matplotlib made easy
MIT License
725 stars 20 forks source link

:bug: fix incompatible of `MatplotlibDeprecationWarning` #31

Closed frostime closed 9 months ago

frostime commented 9 months ago

After matplotlib v2.7.0 updated, the matplotlib.cbook.MatplotlibDeprecationWarning was deprecated, see https://matplotlib.org/stable/api/prev_api_changes/api_changes_3.7.0.html#deprecation-aliases-in-cbook

In order to avoid downstream breakage, these aliases will now warn, and their removal has been pushed from 3.6 to 3.8 to give time to notice said warnings. As replacement, please use matplotlib.MatplotlibDeprecationWarning.

This pull request, simply replaced mpl.cbook.MatplotlibDeprecationWarning with new came mpl.MatplotlibDeprecationWarning to prevent error.

Kislovskiy commented 7 months ago

@lgienapp are you planning to publish a new release with this fix? 🤓

lgienapp commented 7 months ago

Ah, yes, thanks for the reminder; that should've gone through automatically after the merge, but there was an error in the pypi config. Is fixed now and v0.0.6 with your PR is live on pypi. Thanks again for the contribution!