warm_start for SGD#75

Open

jylim2016 wants to merge 1 commit intoibayer:masterfrom

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