search

vkantor / MIPT_Data_Mining_In_Action_2016

"Data Mining in Action Course", Moscow Institute of Physics and Technologies
209 stars 175 forks source link

from __future__ import division #30

Open dmylnikov opened 7 years ago

dmylnikov commented 7 years ago

Может стоит добавлять эту строчку в файлы домашки? а то люди, выросшие на python 3 мучаются, не понимая, почему у них какой-нибудь check_grad() не сходится.

senya-ashukha commented 7 years ago

У нас дз на питон 2!

On Tue, Nov 1, 2016, 12:11 AM Dmitry Mylnikov notifications@github.com wrote:

Может стоит добавлять эту строчку в файлы домашки? а то люди, выросшие на python 3 мучаются, не понимая, почему у них какой-нибудь check_grad() не сходится.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/vkantor/MIPT_Data_Mining_In_Action_2016/issues/30, or mute the thread https://github.com/notifications/unsubscribe-auth/AKBNUyy8ZQKJM0ubkuSNo4vxcdaPgZhuks5q5lmXgaJpZM4KlgqK .

Yours sincerely, Ashuha Arseniy.

dmylnikov commented 7 years ago

кстати, в logreg.ipnb def viz_opt(func, gradf, X, y, n_iter=10, lr=1e-3): w = optimize.fmin_l_bfgs_b(func, w, approx_grad=grad)[0] градиент не в тот параметр вставлен, хотя это и не влияет на результат в данном случае

kirillbobyrev commented 7 years ago

@dmylnikov ну, если уж совсем по-хорошему делать, то вообще надо не только это импортить из future. Плюс ещё six.moves использовать.