[Replacement]: `compute-dot`

[gameroman]

Package to replace

compute-dot

Suggested replacement(s)

const dotProduct = (vector1, vector2) => vector1.reduce((sum, n, i) => sum + n * vector2[i], 0)

Manifest type

micro-utility (tiny utility replaceable with native code or removal)

Rationale

simple util

Availability

No response

Code example (optional)

[kgryte]

Your inline suggestion is not a drop-in replacement, as compute-dot (a) supports an accessor function and (b) avoids reduce call overhead. Furthermore, the dot product of empty vectors should not be 0.

[gameroman]

It is 0 mathematically, though you are write about performance 👍

[kgryte]

The use of the additive identity is a choice, and a common one given the algebraic definition, but one should recognize that the dot product of empty vectors does not have a particularly useful geometric interpretation, as any theta can satisfy the relation. This is especially true when, say, computing the cosine similarity where returning 0 can be misleading.