Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(613)

Unified Diff: components/translate/core/browser/translate_ranker_impl.cc

Issue 2803353002: Change dest_language to target_language in the logistic regression model for consistency with googl… (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/translate/core/browser/translate_ranker_impl.cc
diff --git a/components/translate/core/browser/translate_ranker_impl.cc b/components/translate/core/browser/translate_ranker_impl.cc
index 40fd850ff0acdc3ce6055475b0ada9c53cf148b1..04abc20b333ea36f5d6c89bdb35f486d28ad8ea5 100644
--- a/components/translate/core/browser/translate_ranker_impl.cc
+++ b/components/translate/core/browser/translate_ranker_impl.cc
@@ -259,7 +259,7 @@ double TranslateRankerImpl::CalculateScore(
(features.denied_ratio * logit.decline_ratio_weight()) +
(features.ignored_ratio * logit.ignore_ratio_weight()) +
ScoreComponent(logit.source_language_weight(), features.src_lang) +
- ScoreComponent(logit.dest_language_weight(), features.dst_lang) +
+ ScoreComponent(logit.target_language_weight(), features.dst_lang) +
ScoreComponent(logit.country_weight(), features.country) +
ScoreComponent(logit.locale_weight(), features.app_locale);

Powered by Google App Engine
This is Rietveld 408576698