Skip to content
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

[FEATURE] Switch scipy sparse matrix to scipy sparse array #2090

Open
SimonYansenZhao opened this issue Apr 23, 2024 · 1 comment
Open

[FEATURE] Switch scipy sparse matrix to scipy sparse array #2090

SimonYansenZhao opened this issue Apr 23, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@SimonYansenZhao
Copy link
Collaborator

Description

As mentioned in the doc of scipy.sparse, the scipy sparse matrix interface will be deprecated and replaced by the scipy sparse array interface. All code in Recommenders now use the scipy sparse matrix interface. This issue may cause problems in the future if there is incompatibility or when the sparse matrix interface is completely deprecated.

Expected behavior with the suggested feature

All scipy sparse matrix code should be replaced by code using scipy sparse array.

Other Comments

@SimonYansenZhao SimonYansenZhao added the enhancement New feature or request label Apr 23, 2024
@miguelgfierro
Copy link
Collaborator

@SimonYansenZhao is this fixed with the latest PR #2083

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants