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

scipy.sparse.[hv]stack don't respect anymore the dtype parameter #3111

Closed
nouiz opened this issue Dec 3, 2013 · 1 comment
Closed

scipy.sparse.[hv]stack don't respect anymore the dtype parameter #3111

nouiz opened this issue Dec 3, 2013 · 1 comment
Milestone

Comments

@nouiz
Copy link

nouiz commented Dec 3, 2013

Hi,

As reported on the mailing list[1], scipy.sparse.[hv]stack do not respect the dtype parameter anymore.

@pv told that it was probably the new fast path added in master recently for those function that cause this.

Here is code that replicate this:

import scipy.sparse
from scipy.sparse import csr_matrix
import numpy as np
x = np.arange(5*5).reshape(5,5)
y = csr_matrix(x)
res = scipy.sparse.vstack([y,y], dtype='float32')
assert res.dtype == "float32"

[1] http://mail.scipy.org/pipermail/scipy-user/2013-December/035197.html

perimosocordiae added a commit to perimosocordiae/scipy that referenced this issue Dec 4, 2013
It is indeed the fast-path change that broke dtype behavior.
perimosocordiae added a commit to perimosocordiae/scipy that referenced this issue Dec 4, 2013
@pv pv closed this as completed in ccde60f Dec 4, 2013
@nouiz
Copy link
Author

nouiz commented Dec 4, 2013

thanks for the quick fix!

andrenarchy pushed a commit to andrenarchy/scipy that referenced this issue Dec 4, 2013
It is indeed the fast-path change that broke dtype behavior.
andrenarchy pushed a commit to andrenarchy/scipy that referenced this issue Dec 4, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant