-
Notifications
You must be signed in to change notification settings - Fork 75
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
Upgrade ndarray and other deps #379
base: master
Are you sure you want to change the base?
Conversation
sorry, I forget to update tests file. I will fix it later today |
Fixed, Let's rerun CI |
hi - I was here to press the ci button but I was wondering if @termoshtt wants to chime in too |
It seems like we should bump MSRV. Or limit the version of clap. Which one would you like? |
Hi - any update on this. This seems to be holding back ndarray updates downstream? |
@@ -37,7 +37,7 @@ rand = "0.8.3" | |||
thiserror = "1.0.24" | |||
|
|||
[dependencies.ndarray] | |||
version = "0.15.2" | |||
version = "0.16.0" |
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.
version = "0.16.0" | |
version = ">=0.15.0, <0.17" |
This would make it easier to use as a library downstream
Bump ndarray from 0.15.6 to 0.16.0
Bump approx from 0.4.0 to 0.5
Bump criterion from 0.3.4 to 0.5.1