superclocks / ffm

update the libffm-1.13. Add FTRL and linear
9 stars 5 forks source link

as for 2 version ffm(ftrl), which is better or maybe one is not right implemented #1

Open jackyhawk opened 3 years ago

jackyhawk commented 3 years ago

https://github.com/superclocks/ffm/blob/master/libffm-ftrl-1.13/ffm.cpp ffm_long align0 = (ffm_long)model.k*3;

vs https://github.com/CNevd/libffm-ftrl/blob/master/ffm.cpp

_ffm_long align0 = (ffm_long)model.k*2;_

or maybe one is not right implemented?

thanks very much

jackyhawk commented 3 years ago

any suggestion?