-
Notifications
You must be signed in to change notification settings - Fork 38
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
Implement broadcast styles #58
Comments
That would be great. It would be good to codify why |
Hmm, it's not though. For example, I will do a review of all existing |
Following julia> (x -> x + rand()).(spzeros(3,3))
3×3 SparseMatrixCSC{Float64,Int64} with 9 stored entries:
[1, 1] = 0.32463
[2, 1] = 0.32463
[3, 1] = 0.32463
[1, 2] = 0.32463
[2, 2] = 0.32463
[3, 2] = 0.32463
[1, 3] = 0.32463
[2, 3] = 0.32463
[3, 3] = 0.32463 |
See also discussion #29 where the current behaviour was decided on: julia> (x -> x + rand()).(Zeros(5))
5-element Fill{Float64,1,Tuple{Base.OneTo{Int64}}}:
0.4797461797606011
0.4797461797606011
0.4797461797606011
0.4797461797606011
0.4797461797606011 |
Well never mind then! That makes things simple and I think your reasoning is correct. |
There's been talk of a
FillStyle
before, andRectDiagonal
could implement Base'sStructuredMatrixStyle
.I've been working on broadcast styles for Nabla.jl so I might take a crack at them for this package in my spare time.
The text was updated successfully, but these errors were encountered: