Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alleviating the dependency conflict between scipy and cartopy #451

Merged
merged 9 commits into from
Jul 5, 2023

Conversation

fzhu2e
Copy link
Collaborator

@fzhu2e fzhu2e commented Jul 4, 2023

Related issue: #450 (comment)

Key Updates:

  • setup.py: removing unnecessary items: matplotlib, scipy, numpy, which are dependencies of seaborn, statsmodels, etc.
  • environment.yml: removing unnecessary items. Now the CI is also faster.
  • wavelet.py: np.int is causing error with the latest numpy; replaced with int.
  • installation.rst: emphasizing the importance of conda install statsmodels before pip install pyleoclim to avoid scipy v1.11.

@fzhu2e fzhu2e requested review from khider and CommonClimate July 4, 2023 21:41
@CommonClimate CommonClimate merged commit 4c09c8d into LinkedEarth:master Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants