Argument u is never used in recommend() method in NearestNeighboursScorer class

Open slavafive opened this issue 3 years ago • 1 comments

https://github.com/benfred/implicit/blob/d57b0b6f4dcd98c2a170de4c0bcd40122c71cdf2/implicit/_nearest_neighbours.pyx#L52 I wonder what is the point of passing argument u in this case since it is not used in this method. This method is called on 77-th line in ItemItemRecommender where we recommend items to specific user denoted by userid.

Aug 27 '22 08:08 slavafive

The u param was necessary in this method at one point - but isn't now after the changes in https://github.com/benfred/implicit/pull/526 .

We could drop this parameter now I guess - this class isn't meant to be part of the user facing API, and people shouldn't be depending directly on it directly.

Sep 10 '22 00:09 benfred