Tiny issue in ClassificationData::spiltDataIntoKFolds()

nickgillian / grt

gesture recognition toolkit

859 stars 285 forks source link

Tiny issue in ClassificationData::spiltDataIntoKFolds() #80

Closed daniellocust closed 8 years ago

daniellocust commented 8 years ago

[problem] At the start of the function: //K can not be zero if( K > totalNumSamples ){ errorLog << "spiltDataIntoKFolds(const UINT K,const bool useStratifiedSampling) - K can not be zero!" << std::endl; return false; } is to compare K and zero, but here is totalNumSamples.

[Suggestion] change the totalNumSamples to 0 to fix the problem.

nickgillian commented 8 years ago

Good catch! I've pushed a fix to this to the dev branch and will merge this into the master branch this weekend with some other updates.