Dev #1
Dev #1
Conversation
| @@ -853,19 +971,25 @@ | |||
| " \"\"\"\n", | |||
| " \n", | |||
| " ### START CODE HERE ### (≈ 1 line of code)\n", | |||
| " loss = None\n", | |||
| " loss = np.sum(abs(y-yhat))\n", | |||
There was a problem hiding this comment.
Возможно тут лучше явно обращаться к функциям numpy
loss = np.sum(np.abs(y-yhat))
потому что "Explicit is better than implicit" https://www.python.org/dev/peps/pep-0020/#id3
Хотя этот код тоже будет правильно работать
https://stackoverflow.com/questions/20953127/why-built-in-functions-like-abs-works-on-numpy-array
There was a problem hiding this comment.
Внес правки и добавил np.abs
Никогда не задумывался, какая разница между np.abs и abs ...
| "\n", | ||
| " # Gradient descent (≈ 1 line of code)\n", | ||
| " parameters, grads, costs = None\n", | ||
| " parameters, grads, costs = optimize(w, b, X_train, Y_train, num_iterations, learning_rate, print_cost = False)\n", |
There was a problem hiding this comment.
Не стоит тут хардкодить print_cost = False. Лучше в optimize() пробросить значение print_cost, которое принимает model(), иначе теряется смысл передавать гиперпараметр print_cost в метод model()
There was a problem hiding this comment.
Согласен с замечанием, добавил необходимые правки в "model"
|
great! |
Realization: