fastFM icon indicating copy to clipboard operation
fastFM copied to clipboard

warm_start for SGD

Open jylim2016 opened this issue 8 years ago • 2 comments

jylim2016 avatar Nov 04 '16 18:11 jylim2016

@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

ibayer avatar Nov 22 '16 12:11 ibayer

#74

ibayer avatar Nov 27 '16 14:11 ibayer