LxMLS / lxmls-toolkit

Machine Learning applied to Natural Language Processing Toolkit used in the Lisbon Machine Learning Summer School
Other
223 stars 215 forks source link

Mira algorithm step size #119

Closed tatyana-ruzsics closed 5 years ago

tatyana-ruzsics commented 6 years ago

Mira with regularizer=1 gives no update to the parameters for the Amazon dataset.. It looked better when we've changed the floor division '//' to '/' in the stepsize formula. Was there probably a thought behind using the floor division?

ramon-astudillo commented 5 years ago

Solved by https://github.com/LxMLS/lxmls-toolkit/pull/149