MNT Remove DeprecationWarning for scipy.sparse.linalg.cg tol vs rtol argument#26814
MNT Remove DeprecationWarning for scipy.sparse.linalg.cg tol vs rtol argument#26814lesteve merged 12 commits intoscikit-learn:mainfrom
Conversation
❌ Linting issuesThere was an issue running the linter job. Please update with |
|
OK there is still one issue with |
thomasjpfan
left a comment
There was a problem hiding this comment.
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! |
thomasjpfan
left a comment
There was a problem hiding this comment.
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: |
|
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 ...