Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add option to drop P for ALS #126

Closed
mdekstrand opened this issue Nov 21, 2019 · 2 comments · Fixed by #209
Closed

Add option to drop P for ALS #126

mdekstrand opened this issue Nov 21, 2019 · 2 comments · Fixed by #209
Assignees
@carlos10seg
Labels
algorithms LensKit algorithm implementations
Milestone
0.12.0

Comments

@mdekstrand
Copy link
Member

Once we have #114, there may be use cases in which we always have new ratings. If runtime user training is fast enough, then we can save memory by not storing the user-feature matrix in memory.

This should be controlled with a save_user_features parameter to the constructors that defaults to True.
@mdekstrand
Copy link
Member Author

This is a specific case of #141.

@mdekstrand mdekstrand mentioned this issue Jun 2, 2020
Closed
3 tasks
@mdekstrand mdekstrand added the algorithms LensKit algorithm implementations label Jun 10, 2020
@mdekstrand
Copy link
Member Author

Now that #114 is done, we are ready for this.

@mdekstrand mdekstrand linked a pull request Nov 24, 2020 that will close this issue
Add option to drop user features after training ALS models #209
Merged
@mdekstrand mdekstrand added this to the 0.12.0 milestone Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlos10seg carlos10seg

Labels
algorithms LensKit algorithm implementations
Projects
None yet
Milestone
0.12.0
Development

Successfully merging a pull request may close this issue.

Add option to drop user features after training ALS models carlos10seg/lkpy
2 participants
@mdekstrand @carlos10seg