gesistsa / quanteda.proximity

📐 Proximity-based Weighting Scheme for the Quantitative Analysis of Textual Data
GNU General Public License v3.0
4 stars 0 forks source link

Alternatives to row_mins_c #37

Open kbenoit opened 10 months ago

kbenoit commented 10 months ago

https://stackoverflow.com/a/69195111/4158274

matrixStats::rowMins()

and what about ties?

schochastics commented 10 months ago

row_mins_c is giving the same results as matrixStats::rowMins() but is slightly faster because it is optimized for our case. One could surely opt for matrixStats::rowMins() but that would add a dependency that is not necessary.

Concerning ties: We only need the minimum value in a row, not its index (but @chainsawriot sould confirm that)

chainsawriot commented 10 months ago

@schochastics Regarding ties: Yes, we only need the minimum value.