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

Fix for numpy 1.20 and set numpy>=1.16 #2616

Merged
merged 9 commits into from Jan 31, 2021

Conversation

ericpre
Copy link
Member

@ericpre ericpre commented Jan 6, 2021

In line with NEP 29 Sympy have dropped support for the numpy<1.16 in their latest release (sympy 1.7.1) and the oldest supported version CI build is failing accordingly. More libraries are going to drop support for numpy <1.16 soon and we can do the same and tidy up some code required to support numpy <1.16.

Progress of the PR

… that numpy<1.16 are not supported anymore and we can rely on the __array_function__ protocol.
@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #2616 (da54ab5) into RELEASE_next_patch (5576d48) will decrease coverage by 0.02%.
The diff coverage is 91.30%.

Impacted file tree graph

@@                  Coverage Diff                   @@
##           RELEASE_next_patch    #2616      +/-   ##
======================================================
- Coverage               76.33%   76.30%   -0.03%     
======================================================
  Files                     202      202              
  Lines                   29640    29601      -39     
  Branches                 6469     6461       -8     
======================================================
- Hits                    22625    22588      -37     
+ Misses                   5237     5236       -1     
+ Partials                 1778     1777       -1     
Impacted Files Coverage Δ
hyperspy/io_plugins/protochips.py 95.34% <ø> (-0.08%) ⬇️
hyperspy/signal.py 75.84% <77.77%> (-0.05%) ⬇️
hyperspy/_components/gaussian.py 97.10% <100.00%> (-0.27%) ⬇️
hyperspy/_components/lorentzian.py 97.05% <100.00%> (-0.35%) ⬇️
hyperspy/_components/skew_normal.py 88.77% <100.00%> (-0.95%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5576d48...6ec01f9. Read the comment docs.

Copy link
Contributor

@jlaehne jlaehne left a comment

Choose a reason for hiding this comment

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

LGTM

@ericpre ericpre changed the title Numpy 1.16 Fix for numpy 1.20 and set numpy>=1.16 Jan 31, 2021
@thomasaarholt
Copy link
Contributor

Great stuff, LGTM

@thomasaarholt thomasaarholt merged commit 66bc094 into hyperspy:RELEASE_next_patch Jan 31, 2021
@ericpre ericpre added this to the v1.6.2 milestone Feb 2, 2021
@ericpre ericpre deleted the numpy_1.16 branch February 2, 2021 14:06
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.

None yet

3 participants