You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Your code uses x .+= y, so you should know that in Julia 0.5 this has changed meaning to be equivalent to broadcast!(identity, x, x .+ y), so that it mutates the x array (see JuliaLang/julia#17510 … in Julia 0.6 the whole operation will occur in-place without temporaries). So .+ should only be used if the left-hand side is a mutable array, and you don't mind mutating it.
At first glance, this looks like it is okay for you, because you use it in m.x_counts[c] .+= X[:, j], where it looks like you want to work in-place anyway? But if it were a problem you could always change it to +=.
The text was updated successfully, but these errors were encountered:
Thanks for reporting! Yes, in this case mutating lhs array is exactly what was intended, so changes in 0.5 and 0.6 will even make this code better, without any modifications 👍
Your code uses
x .+= y
, so you should know that in Julia 0.5 this has changed meaning to be equivalent tobroadcast!(identity, x, x .+ y)
, so that it mutates thex
array (see JuliaLang/julia#17510 … in Julia 0.6 the whole operation will occur in-place without temporaries). So.+
should only be used if the left-hand side is a mutable array, and you don't mind mutating it.At first glance, this looks like it is okay for you, because you use it in
m.x_counts[c] .+= X[:, j]
, where it looks like you want to work in-place anyway? But if it were a problem you could always change it to+=
.The text was updated successfully, but these errors were encountered: