-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
DEP: improve message for deprecated private API for sparse
#19226
Conversation
…I in the sparse submodule
TST: fix test for sparse.linalg
sparse
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 @hoangle96 some minor changes required
scipy/_lib/tests/test_public_api.py
Outdated
('scipy.stats.biasedurn', None), | ||
('scipy.stats.kde', None), | ||
('scipy.stats.morestats', None), | ||
('scipy.stats.mstats_basic', 'mstats'), | ||
('scipy.stats.mstats_extras', 'mstats'), | ||
('scipy.stats.mvn', None), | ||
('scipy.stats.stats', None)]) | ||
|
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.
scipy/sparse/bsr.py
Outdated
return getattr(_bsr, name) | ||
return _sub_module_deprecation(sub_package="sparse", module="bsr", | ||
private_modules=["_bsr"], all=__all__, | ||
attribute=name) |
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.
New line at end of file needed
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.
Still missing an new line
scipy/sparse/data.py
Outdated
return getattr(_data, name) | ||
return _sub_module_deprecation(sub_package="sparse", module="data", | ||
private_modules=["_data"], all=__all__, | ||
attribute=name) |
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.
new line at end of file
scipy/sparse/linalg/eigen.py
Outdated
'ArpackError', | ||
'ArpackNoConvergence', | ||
'eigs', | ||
'eigsh', 'lobpcg', 'svds', 'arpack', 'test' |
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.
can you revert the change here please
scipy/sparse/linalg/isolve.py
Outdated
'bicg', | ||
'bicgstab', | ||
'cg', | ||
'cgs', | ||
'gcrotmk', | ||
'gmres', | ||
'lgmres', | ||
'lsmr', | ||
'lsqr', | ||
'minres', | ||
'qmr', | ||
'tfqmr', | ||
'utils', | ||
'iterative', | ||
'test' |
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.
and here
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.
again please revert this change it makes it harder to tell if anything has been removed
scipy/sparse/linalg/matfuncs.py
Outdated
'expm', | ||
'inv', | ||
'solve', | ||
'solve_triangular', | ||
'spsolve', | ||
'is_pydata_spmatrix', | ||
'LinearOperator', | ||
'UPPER_TRIANGULAR', | ||
'MatrixPowerOperator', | ||
'ProductOperator' |
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.
and here
scipy/sparse/linalg/eigen.py
Outdated
return _sub_module_deprecation(sub_package="sparse.linalg", module="eigen", | ||
private_modules=["_eigen"], all=__all__, | ||
attribute=name) | ||
|
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.
one too many lines at end of file!
sparse
sparse
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.
Making sure changes are compliant with PEP 8. Removing unnecessary changes
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 @hoangle96 still a few changes required
scipy/sparse/bsr.py
Outdated
return getattr(_bsr, name) | ||
return _sub_module_deprecation(sub_package="sparse", module="bsr", | ||
private_modules=["_bsr"], all=__all__, | ||
attribute=name) |
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.
Still missing an new line
scipy/sparse/linalg/isolve.py
Outdated
'bicg', | ||
'bicgstab', | ||
'cg', | ||
'cgs', | ||
'gcrotmk', | ||
'gmres', | ||
'lgmres', | ||
'lsmr', | ||
'lsqr', | ||
'minres', | ||
'qmr', | ||
'tfqmr', | ||
'utils', | ||
'iterative', | ||
'test' |
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.
again please revert this change it makes it harder to tell if anything has been removed
@j-bowhay I have updated the changes to be PEP8 complaint and revert the unnecessary changes. |
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 @hoangle96! Congratulations on your first contribution to SciPy, if you are interested in doing one of the other modules from the issue that would be greatly appreciated!
Reference issue
Solving a part of issue gh-18279
closes #19241
What does this implement/fix?
Finish #18279 for
sparse
Additional information