ALS int-String reverse ID mapping conflates user and item IDs
srowen opened this issue · comments
Sean Owen commented
ALSUpdate
maintains a reverse mapping from ints used in the ALS algorithm to IDs. However it mixes user and item IDs in this mapping making it possible for a user ID to suddenly be mapped back to an item ID. Bad.
Sean Owen commented
Fixed, at least, the problem of overlapping user/item IDs.