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

warm_start for SGD #75

Open

jylim2016 wants to merge 1 commit into ibayer:master from jylim2016:patch-1

jylim2016 commented Nov 4, 2016

No description provided.


          warm_start for SGD

bbaf981

Owner

ibayer commented Nov 22, 2016

@jylim2016
Thanks for opening the PR.
I think we need the following before merging:

The warm-start parameter should be added to
https://github.com/ibayer/fastFM/blob/master/fastFM/sgd.py#L73 and
https://github.com/ibayer/fastFM/blob/master/fastFM/sgd.py#L156
A test case similar to the one for the ALS solver
https://github.com/ibayer/fastFM/blob/master/fastFM/tests/test_als.py#L96

Owner

ibayer commented Nov 27, 2016

ibayer mentioned this pull request

BPR FMRecommender #93

Open

ibayer mentioned this pull request

Need partial_fit #148

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet