Class: SocialMF
Method: IterateBatch
Issue:
I think the statement ''neg_trust_times_reg = -social_regularization * trust_v'' is incorrect, it should be "-social_regularization/user_reverse_connections[u].Count" rather than "user_connections[v].Count" which is used in the computation of user w not user v, according to the formula.
Class: SocialMF
Method: IterateBatch
Issue:
I think the statement ''neg_trust_times_reg = -social_regularization * trust_v'' is incorrect, it should be "-social_regularization/user_reverse_connections[u].Count" rather than "user_connections[v].Count" which is used in the computation of user w not user v, according to the formula.