Skip to content

Change model.fit to take a user_items sparse matrix - #484

Merged
benfred merged 1 commit into
api_refactorfrom
model_fit_user_items
Oct 16, 2021
Merged

benfred merged 1 commit into
api_refactorfrom
model_fit_user_items

Conversation

@benfred

@benfred benfred commented Oct 16, 2021

Copy link
Copy Markdown
Owner

Previously, model.fit took an item_users sparse matrix, while model.recommend took a user_items
sparse matrix. This was a source of confusion, so change the model.fit method to be consistent
with model.recommend

 Previously, model.fit took an item_users sparse matrix, while model.recommend took a user_items
 sparse matrix.  This was a source of confusion, so change the model.fit method to be consistent
 with model.recommend
@benfred benfred mentioned this pull request Oct 16, 2021
6 tasks done
@benfred
benfred merged commit ffb490d into api_refactor Oct 16, 2021
@benfred
benfred deleted the model_fit_user_items branch October 16, 2021 17:05
@benfred benfred linked an issue Oct 16, 2021 that may be closed by this pull request
6 tasks done
@benfred benfred removed a link to an issue Nov 26, 2021
6 tasks done
benfred added a commit that referenced this pull request Dec 31, 2021
Previously, model.fit took an item_users sparse matrix, while model.recommend took a user_items
 sparse matrix.  This was a source of confusion, so change the model.fit method to be consistent
 with model.recommend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant