Skip to content

Fix matplotlib deprecation warning#6072

Merged
dstrain115 merged 3 commits intoquantumlib:masterfrom
dstrain115:fix_mpl_deprectation
Apr 21, 2023
Merged

Fix matplotlib deprecation warning#6072
dstrain115 merged 3 commits intoquantumlib:masterfrom
dstrain115:fix_mpl_deprectation

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

No description provided.

@dstrain115 dstrain115 requested review from a team, cduck and vtomole as code owners April 20, 2023 22:43
@dstrain115 dstrain115 requested a review from viathor April 20, 2023 22:43
@CirqBot CirqBot added the Size: XS <10 lines changed label Apr 20, 2023
Comment thread cirq-core/cirq/vis/heatmap_test.py Outdated
_, mesh = random_heatmap.plot(ax)

colormap = mpl.cm.get_cmap(colormap_name)
colormap = mpl.colormaps.get_cmap(colormap_name)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Optional: Issue matplotlib/matplotlib#20853 suggests we should perhaps just say

colormap = mpl.colormaps[colormap_name]

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Maybe this will work in python 3.7 too.

@dstrain115 dstrain115 added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Apr 21, 2023
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Apr 21, 2023
@dstrain115 dstrain115 merged commit 03405d1 into quantumlib:master Apr 21, 2023
@dstrain115 dstrain115 removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Apr 21, 2023
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* Fix matplotlib deprecation warning

* Fix for better deprecation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants