GRA-191: implement pool and crossEntropyLoss#80
GRA-191: implement pool and crossEntropyLoss#80Maxon081102 wants to merge 1 commit intotask33_addOpfrom
Conversation
|
У тебя там тесты не проходят, ты там forward не убрал |
| // WARNING: если проблемы, меняем на случай с EPS | ||
| return - 1.0f / (a(k, l, i, j)); | ||
| // return - 1.0f / (a(k, l, i, j) + EPS); |
There was a problem hiding this comment.
так а может чтоб проблем не было сразу на EPS поставим, ахахах, не?
There was a problem hiding this comment.
нет, у torch такого нету
| } | ||
| }; | ||
|
|
||
| class LazyBlobEntropyDerivative: public LazyBlob { |
There was a problem hiding this comment.
даа, ну совсем понесло, теперь уже и производную запихиваем в, вообще то подразумевавшиеся как элементарные операции
There was a problem hiding this comment.
а производную по-другому не посчитать, очень тяжелая операция
| Blob Exp::compute(const vector<LazyBlobRef>& args) const { | ||
| args1(a); | ||
| return a.applying([](float x) { return std::exp(x); }); | ||
| } |
There was a problem hiding this comment.
а зато вот экспоната это теперь операция....
капец совсем бедную архитектуру разворатили
There was a problem hiding this comment.
а что это еще должно быть, как не операция?
Maxon081102
force-pushed
the
task191_entropy_pool
branch
from
724fdb1 to
b1c44e9
Compare
Artem-Goldenberg
left a comment
Artem-Goldenberg
left a comment
There was a problem hiding this comment.
Ладно все норм, я правда не знаю куда это уже вливать ....
2 participants
https://ru.yougile.com/team/b400e1850fe9/#GRA-191