Skip to content

Fix Matplotlib, ipyparallel and dict deprecation warnings#202

Merged
EwoutH merged 4 commits intomasterfrom
depreciations-2022-10
Oct 25, 2022
Merged

Fix Matplotlib, ipyparallel and dict deprecation warnings#202
EwoutH merged 4 commits intomasterfrom
depreciations-2022-10

Conversation

@EwoutH
Copy link
Copy Markdown
Collaborator

@EwoutH EwoutH commented Oct 23, 2022

Maintenance in it's purest form. This PR fixed 4 types of deprecations:

  • Fix MarkerStyle(None) deprecation in matplotlib by replacing it with MarkerStyle('')
  • Replace deprecated .iteritems() with .items() for dicts
  • Fix Matplotlib deprecation of loc as a positional keyword in legend functions
  • Import launcher from ipyparallel.cluster instead of ipyparallel.apps

I founds some others that might be less trivial, so I opened tracking issue (#201).

@EwoutH EwoutH added this to the 2.3.0 milestone Oct 23, 2022
@EwoutH EwoutH requested a review from quaquel October 23, 2022 22:03
@EwoutH EwoutH mentioned this pull request Oct 23, 2022
12 tasks
@coveralls
Copy link
Copy Markdown

coveralls commented Oct 23, 2022

Coverage Status

Coverage remained the same at 81.139% when pulling ef9ecf7 on depreciations-2022-10 into 3c1cf7b on master.

@EwoutH EwoutH force-pushed the depreciations-2022-10 branch from 2f797a3 to f5c8220 Compare October 23, 2022 22:06
@EwoutH
Copy link
Copy Markdown
Collaborator Author

EwoutH commented Oct 25, 2022

Note to self: Do fast-forward merge

Fix two cases of the following deprecation:

MatplotlibDeprecationWarning: MarkerStyle(None) is deprecated since 3.6; support will be removed two minor releases later.  Use MarkerStyle('') to construct an empty MarkerStyle.

See matplotlib/matplotlib#21074
Replace 7 cases of the deprecated .iteritems() (from Python 2) being called, with the current items().
With the 3.6 release Matplotlib deprecated using `loc` as a positional keyword for the figure.legend function, making it a keyword only function starting from Matplotlib 3.8.

This commit adds the `loc` keyword to the figure.legend and figlegend functions to resolve the depreciation warning and prepare for future Matplotlib versions.
DeprecationWarning: ipyparallel.apps.launcher is deprecated in ipyparallel 7. Use ipyparallel.cluster.launcher.

See: ipython/ipyparallel#463
@EwoutH EwoutH force-pushed the depreciations-2022-10 branch from f5c8220 to ef9ecf7 Compare October 25, 2022 17:16
@EwoutH EwoutH merged commit ef9ecf7 into master Oct 25, 2022
@quaquel quaquel deleted the depreciations-2022-10 branch November 10, 2022 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants