DeepPavlov
DeepPavlov copied to clipboard
Faq output correction
Для корректной работы следует поменять 2 конфига - tfidf_logreg_autofaq и tfidf_logreg_en. Если изменение конфига это прямо не ок, то можем сделать наше изменение в виде отдельной ноды, чтобы можно было ее просто добавлять в chainer в кастомных конфигах. Вот)
Изменение парочки конфигов - это ок, изменение выхода компоненты proba2labels
- не ок. она универсальная в данных момент и много где используется, если менять ее - придется менять все конфиги.
Изменение парочки конфигов - это ок, изменение выхода компоненты
proba2labels
- не ок. она универсальная в данных момент и много где используется, если менять ее - придется менять все конфиги.
Здравствуйте!
Нам показалось, что единственное логичное место, где можно обрезать top_n (или выбрать максимальное) - это в этой компоненте, поскольку только она получает одновременно индексы и вероятности. Во всех конфигах, где используется proba2labels нужно менять "out" - это факт, но мы можем сделать это самостоятельно. Но при этом в нашем коде не поменялась сама идеология поведения, а поменялся только формат вывода. Если есть предложения, как это сделать по-другому без изменения конфигов - мы можем поправить. Есть, например, идея с передачей атрибута и использовать его как флаг вывода, но нам кажется это не самое красивое решение. Другая идея - вставить новую ноду, которая ничего не делает, кроме обрезки списка вероятностей, но опять же - не элегантно как-то :)