Fix: Probability used for one word contextual phrase.
Bug: 14161647 Change-Id: Ia5f985ea467145fb381e3ad2c1f67f3b013f19e8
This commit is contained in:
parent
6aa64e91ed
commit
cb368a6dac
1 changed files with 16 additions and 17 deletions
|
@ -622,9 +622,7 @@ public class DictionaryFacilitator {
|
||||||
}
|
}
|
||||||
PrevWordsInfo prevWordsInfo = PrevWordsInfo.BEGINNING_OF_SENTENCE;
|
PrevWordsInfo prevWordsInfo = PrevWordsInfo.BEGINNING_OF_SENTENCE;
|
||||||
for (int i = 0; i < phrase.length; i++) {
|
for (int i = 0; i < phrase.length; i++) {
|
||||||
if (i < phrase.length - 1) {
|
final String[] subPhrase = Arrays.copyOfRange(phrase, i /* start */, phrase.length);
|
||||||
final String[] subPhrase =
|
|
||||||
Arrays.copyOfRange(phrase, i /* start */, phrase.length);
|
|
||||||
final String subPhraseStr = TextUtils.join(Constants.WORD_SEPARATOR, subPhrase);
|
final String subPhraseStr = TextUtils.join(Constants.WORD_SEPARATOR, subPhrase);
|
||||||
contextualDict.addUnigramEntryWithCheckingDistracter(
|
contextualDict.addUnigramEntryWithCheckingDistracter(
|
||||||
subPhraseStr, probability, null /* shortcutTarget */,
|
subPhraseStr, probability, null /* shortcutTarget */,
|
||||||
|
@ -635,7 +633,7 @@ public class DictionaryFacilitator {
|
||||||
contextualDict.addNgramEntry(prevWordsInfo, subPhraseStr,
|
contextualDict.addNgramEntry(prevWordsInfo, subPhraseStr,
|
||||||
bigramProbabilityForPhrases, BinaryDictionary.NOT_A_VALID_TIMESTAMP);
|
bigramProbabilityForPhrases, BinaryDictionary.NOT_A_VALID_TIMESTAMP);
|
||||||
|
|
||||||
}
|
if (i < phrase.length - 1) {
|
||||||
contextualDict.addUnigramEntryWithCheckingDistracter(
|
contextualDict.addUnigramEntryWithCheckingDistracter(
|
||||||
phrase[i], probability, null /* shortcutTarget */,
|
phrase[i], probability, null /* shortcutTarget */,
|
||||||
Dictionary.NOT_A_PROBABILITY /* shortcutFreq */,
|
Dictionary.NOT_A_PROBABILITY /* shortcutFreq */,
|
||||||
|
@ -644,6 +642,7 @@ public class DictionaryFacilitator {
|
||||||
DistracterFilter.EMPTY_DISTRACTER_FILTER);
|
DistracterFilter.EMPTY_DISTRACTER_FILTER);
|
||||||
contextualDict.addNgramEntry(prevWordsInfo, phrase[i],
|
contextualDict.addNgramEntry(prevWordsInfo, phrase[i],
|
||||||
bigramProbabilityForWords, BinaryDictionary.NOT_A_VALID_TIMESTAMP);
|
bigramProbabilityForWords, BinaryDictionary.NOT_A_VALID_TIMESTAMP);
|
||||||
|
}
|
||||||
prevWordsInfo = new PrevWordsInfo(phrase[i]);
|
prevWordsInfo = new PrevWordsInfo(phrase[i]);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue