Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

update fineobj to 15 47 #1013

Merged
merged 11 commits into from
Dec 25, 2023
Prev Previous commit
Next Next commit
Remove unused variable
Signed-off-by: Valery Fedyunin <valery.fedyunin@abbyy.com>
  • Loading branch information
Valery Fedyunin committed Dec 22, 2023
commit b8ec992c0dbb046cbdb0b24e42fc84d2c8218d03
2 changes: 0 additions & 2 deletions NeoML/src/TraditionalML/UnigramTrainer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,6 @@ void CUnigramTrainer::fillTrainDict( const CWordDictionary& frequencyDict )
void CUnigramTrainer::createInitialVocab()
{
CTrieNode<CTrieCounterData> substringTrie;
int64_t totalCount = 0;

for( int i = 0; i < trainDict.Size(); ++i ) {
const CString& word = trainDict.GetWord( i );
Expand All @@ -225,7 +224,6 @@ void CUnigramTrainer::createInitialVocab()
data.End = end;
}
data.Count += count;
totalCount += count;

if( tokenLength++ == maxSubwordLength ) {
break;
Expand Down