DeepPavlov icon indicating copy to clipboard operation
DeepPavlov copied to clipboard

Faq output correction

Open tazizov opened this issue 4 years ago • 2 comments

Для корректной работы следует поменять 2 конфига - tfidf_logreg_autofaq и tfidf_logreg_en. Если изменение конфига это прямо не ок, то можем сделать наше изменение в виде отдельной ноды, чтобы можно было ее просто добавлять в chainer в кастомных конфигах. Вот)

tazizov avatar Sep 03 '20 10:09 tazizov

Изменение парочки конфигов - это ок, изменение выхода компоненты proba2labels - не ок. она универсальная в данных момент и много где используется, если менять ее - придется менять все конфиги.

dilyararimovna avatar Sep 08 '20 13:09 dilyararimovna

Изменение парочки конфигов - это ок, изменение выхода компоненты proba2labels - не ок. она универсальная в данных момент и много где используется, если менять ее - придется менять все конфиги.

Здравствуйте!

Нам показалось, что единственное логичное место, где можно обрезать top_n (или выбрать максимальное) - это в этой компоненте, поскольку только она получает одновременно индексы и вероятности. Во всех конфигах, где используется proba2labels нужно менять "out" - это факт, но мы можем сделать это самостоятельно. Но при этом в нашем коде не поменялась сама идеология поведения, а поменялся только формат вывода. Если есть предложения, как это сделать по-другому без изменения конфигов - мы можем поправить. Есть, например, идея с передачей атрибута и использовать его как флаг вывода, но нам кажется это не самое красивое решение. Другая идея - вставить новую ноду, которая ничего не делает, кроме обрезки списка вероятностей, но опять же - не элегантно как-то :)

tazizov avatar Sep 08 '20 13:09 tazizov