Deprecate vectorized abs2 methods in favor of compact broadcast syntax #18564
Conversation
base/sparse/sparsevector.jl
Outdated
| @@ -1,4 +1,4 @@ | |||
| # This file is a part of Julia. License is MIT: http://julialang.org/license | |||
| 9# This file is a part of Julia. License is MIT: http://julialang.org/license | |||
kshyatt
added
maths
|
Rebased. Best! |
|
your sparse vector broadcast pr will fix the fusing concern here, right? |
Should, yes. Thanks! |
| broadcast(::typeof(abs), x::AbstractSparseVector) = SparseVector(length(x), copy(nonzeroinds(x)), abs.(nonzeros(x))) | ||
| for op in [:abs2, :conj] | ||
| broadcast(::typeof(abs2), x::AbstractSparseVector) = SparseVector(length(x), copy(nonzeroinds(x)), abs2.(nonzeros(x))) | ||
| for op in [:conj] |
There was a problem hiding this comment.
will conj be implemented as a method of broadcast once that supports sparse vectors?
There was a problem hiding this comment.
IIUC conj methods for vectors/matrices should persist per discussion in #18495 (comment). Thoughts? Thanks!
There was a problem hiding this comment.
right the method will stay without dots, I'm asking if this implementation will remain. not really a feature freeze issue, but possible cleanup for later
There was a problem hiding this comment.
Cheers, with you now. Definitely a candidate for cleanup, and likewise for several more methods in base/sparse/sparsevectors.jl. Best!
|
Subsumed by #19791. |
3 participants
This PR deprecates the last remaining vectorized
abs2method in favor of compact broadcast syntax. Ref. #16285, #17302, #18495, #18512, #18513, and #18558. Best!(Unlike with
float,real, etc., the remaining vectorizedabs2method never aliases its input. This PR should be less controversial than #18495, #18512, and #18513 as a result.)