-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
MNT Remove DeprecationWarning for scipy.sparse.linalg.cg tol vs rtol argument #26814
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
Conversation
❌ Linting issuesThere was an issue running the linter job. Please update with |
OK there is still one issue with |
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.
Looks like this PR is conflicting with #26813
…nto sparse-cg-rtol
I wonder if the new |
I took
Let me know in case you had something else in mind! |
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.
Thank you for the PR! I do not see a better alternative to this PR. LGTM
Note the remaining errors all look like some formatting changes for numpy scalars in numpy development version, better to leave them for a separate PR. This is due to numpy/numpy#22449 that implements NEP 51. For example:
|
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 as well.
I think it is worth to acknowledge the changes in the |
Merging my own PR with two approvals 😅 |
It looks like the remaining failures may be fixed by #27042 |
Fix most of scipy-dev issues seen in #26791
Looking at
https://github.com/scipy/scipy/pull/18488/files#diff-b65f85e6dd5881e4611b34a584512f8df370e520758c30e8d297e70c61dcf48f, it does not seem there is a way to keep the
atol="legacy"
behaviour. This means that potentially there will be a behaviour change for our users ...