-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
loosen remaining core dependency versions #1917
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1917 +/- ##
========================================
Coverage 98.47% 98.47%
========================================
Files 77 79 +2
Lines 3545 3681 +136
========================================
+ Hits 3491 3625 +134
- Misses 54 56 +2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thanks for catching this old issue. I almost forgot about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this old issue!
See minor comment about the scipy version upper bound.
After clarifying this small aspect, LGTM!
46b0237
to
5a7e4a4
Compare
Description
In #1672 we loosened the version requirement for
numpy
and haven't had any complaints from users in regards to irreproducible bugs/inconsistencies.We could do each one of these dependencies at a time, but considering how little we use
scipy
, I think this should be fine to go out as one. Happy to change if others have different opinions.Fixes #1667