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
Currently pairwise works with cor but not with corkendall or corspearman
I think the fix is straightforward: Change the type declarations for each method of corkendall and corspearman to be AbstractVector in place of AbstractVector{<:Real} and AbstractMatrix in place of AbstractMatrix{<:Real}.
If I made a PR for that would it be likely to be merged?
Closing because I think the solution proposed is not a good one. Better to change pairwise so that it calls f passing in x and y of type AbstractVector{<:Real}.
Currently
pairwise
works withcor
but not withcorkendall
orcorspearman
I think the fix is straightforward: Change the type declarations for each method of
corkendall
andcorspearman
to beAbstractVector
in place ofAbstractVector{<:Real}
andAbstractMatrix
in place ofAbstractMatrix{<:Real}
.If I made a PR for that would it be likely to be merged?
The text was updated successfully, but these errors were encountered: