Update usability score thresholds#445
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #445 +/- ##
=======================================
Coverage 73.30% 73.30%
=======================================
Files 443 443
Lines 37203 37203
Branches 5110 5110
=======================================
Hits 27272 27272
Misses 8805 8805
Partials 1126 1126 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
ddaspit
left a comment
There was a problem hiding this comment.
@ddaspit reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on Enkidu93 and pmachapman).
src/SIL.Machine/QualityEstimation/ChrF3QualityEstimator.cs line 35 at r1 (raw file):
/// The threshold values used to calculate the usability label for every book or text. /// </summary> public Thresholds BookThresholds { get; set; } = new Thresholds(greenThreshold: 0.776, yellowThreshold: 0.681);
These values should be constants.
Enkidu93
left a comment
There was a problem hiding this comment.
Will any of the changes here need to be ported as well? Are there plans for silnlp to move to using machine.py for this soon so it's easier to keep the code in sync?
@Enkidu93 reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on pmachapman).
These are the updated values from silnlp.
This change is