Skip to content

pin matplotlib version#2583

Merged
MMathisLab merged 2 commits intomainfrom
niels/pin_matplotlib
Jun 5, 2024
Merged

pin matplotlib version#2583
MMathisLab merged 2 commits intomainfrom
niels/pin_matplotlib

Conversation

@n-poulsen
Copy link
Contributor

@n-poulsen n-poulsen commented May 20, 2024

Addresses issue #2581. Replicated in Google Colab and locally. We use this method in many functions

With matplotlib==3.8.4:

>>> import matplotlib as mpl
>>> print(mpl.__version__)
3.8.4
>>> import matplotlib.pyplot as plt
>>> plt.cm.get_cmap
<function _get_cmap at 0x11e9f4430>

With matplotlib==3.9.0:

>>> import matplotlib as mpl
>>> print(mpl.__version__)
3.9.0
>>> import matplotlib.pyplot as plt
>>> plt.cm.get_cmap
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'matplotlib.cm' has no attribute 'get_cmap'

@n-poulsen n-poulsen added the Another package code issue Bug/issue that relates to a dlc dependency label May 20, 2024
@n-poulsen n-poulsen requested review from AlexEMG and MMathisLab May 20, 2024 08:51
@n-poulsen n-poulsen changed the title Niels/pin matplotlib pin matplotlib version May 20, 2024
@MMathisLab MMathisLab merged commit 0835967 into main Jun 5, 2024
@MMathisLab MMathisLab deleted the niels/pin_matplotlib branch June 5, 2024 09:25
@NadjaBrun
Copy link

NadjaBrun commented Jun 6, 2024

Hi! Thanks for working on this!

I use google colab and just tried to start evaluating. However, the error message remains:
AttributeError: module 'matplotlib.cm' has no attribute 'get_cmap'

Any help is much appreciated!

@MMathisLab
Copy link
Member

MMathisLab commented Jun 6, 2024

yes, matplotlib just changed, so you will need to explicitly call and older version by running in Colab: !pip install matplotlib==(older version) -- or install from source our package !pip install git+https://github.com/DeepLabCut/DeepLabCut.git; DeepLabCut is not updated on pypi only the repo here

@NadjaBrun
Copy link

NadjaBrun commented Jun 6, 2024

The evaluation step worked! Thank you so much!

@boeddeker boeddeker mentioned this pull request Jun 13, 2024
Merged
@smaginmi smaginmi mentioned this pull request Jun 20, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MMathisLab MMathisLab MMathisLab approved these changes

@AlexEMG AlexEMG Awaiting requested review from AlexEMG

Assignees

No one assigned

Labels

Another package code issue Bug/issue that relates to a dlc dependency

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@n-poulsen @NadjaBrun @MMathisLab