Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove normalization of confidence scores in intent classification #782

Merged
merged 2 commits into from
Apr 5, 2019

Conversation

adrienball
Copy link
Contributor

Description:
The logreg intent classifier used to have a specific logic when used with an intents filter: the intent classification scores were renormalized to sum to 1.0.

This could lead to unexpected behaviors, especially when all the intents in the intents filter are associated to very low confidence scores. In such cases, the renormalization would significantly increase the scores.

Checklist:

  • My PR is ready for code review
  • I have added some tests, if applicable, and run the whole test suite, including linting tests
  • I have updated the documentation, if applicable

The logreg intent classifier used to have a specific logic when used
with an intents filter: the intent classification scores were
renormalized to sum to 1.0.

This could lead to unexpected behaviors, especially when all the intents
in the intents filter are associated to very low confidence scores. In such
cases, the renormalization would significantly increase the scores.
@codecov-io
Copy link

Codecov Report

Merging #782 into develop will increase coverage by 0.11%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #782      +/-   ##
===========================================
+ Coverage     88.1%   88.22%   +0.11%     
===========================================
  Files           76       76              
  Lines         4541     4576      +35     
  Branches       871      904      +33     
===========================================
+ Hits          4001     4037      +36     
+ Misses         403      401       -2     
- Partials       137      138       +1

@adrienball adrienball merged commit b3a6bae into develop Apr 5, 2019
@adrienball adrienball deleted the task/remove-score-normalization branch April 5, 2019 15:38
@adrienball adrienball mentioned this pull request Apr 10, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants