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.

Sign up for GitHub

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

Jump to bottom

.* not defined for sparse matrices #11475

Closed
mauro3 opened this issue May 28, 2015 · 5 comments
Closed

.* not defined for sparse matrices #11475

mauro3 opened this issue May 28, 2015 · 5 comments

Comments

@mauro3
Copy link
Contributor

mauro3 commented May 28, 2015 

julia> sp = sprand(3,3,0.1)
3x3 sparse matrix with 1 Float64 entries:
        [2, 2]  =  0.885289

julia> [1,1,1].* sp
ERROR: DimensionMismatch("")
 in .* at sparse/sparsematrix.jl:553
 in .* at sparse/sparsematrix.jl:655

note that broadcasting itself is also broken for sparse matrices: #11474
@NyanHelsing
Copy link

I can't seem to reproduce this.

@JeffBezanson
Copy link
Sponsor Member

Maybe this was fixed for 0.4?

@mauro3
Copy link
Contributor Author

mauro3 commented May 28, 2015

Yes, it is 0.3. And it works as expected on 0.4. Oddly though broadcast makes it full on 0.4: see #11475

@mauro3 mauro3 mentioned this issue May 28, 2015
Closed
@tkelman
Copy link
Contributor

tkelman commented May 29, 2015

I think this was fixed by #10106 which is a bit of a large change to consider backporting. Okay to close?

@mauro3
Copy link
Contributor Author

mauro3 commented May 29, 2015

Yes, the broadcast! workaround of yours in #11474 is good enough.

@mauro3 mauro3 closed this as completed May 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@JeffBezanson @NyanHelsing @mauro3 @tkelman