You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks @jamesmyatt , this is a great catch! Captum only requires matplotlib but not pyqt, we will update the recipe to depend on matplotlib-base instead.
Summary:
This addresses #644 , switching the Conda dependency from matplotlib to matplotlib-base, since pyqt is not used in Captum.
Pull Request resolved: #648
Reviewed By: nlaptev
Differential Revision: D27604398
Pulled By: vivekmig
fbshipit-source-id: 9241a2d26b81d2232fcf801e9742e93b2238d722
According to the conda-forge docs, the conda recipe should probably depend on the matplotlib-base package, rather than matplotlib: https://conda-forge.org/docs/maintainer/knowledge_base.html#matplotlib
Is this right? Does captum require pyqt as well or just matplotlib?
For example: conda-forge/geoplot-feedstock#19
The text was updated successfully, but these errors were encountered: