[MAHOUT-605] Array returned by classifier.bayes.algorithm.CBayesAlgorithm.classifyDocument is sorted ascendant

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Not A Problem
• Affects Version/s: 0.4
• Fix Version/s: 0.5
• Labels: bayesian classification

Environment

Linux

• Assignee: Robin Anil
• Reporter: Robin Swezey
• Due: 11/Feb/11
• Created: 03/Feb/11 12:47
• Updated: 31/Jan/24 22:12
• Resolved: 26/Mar/11 13:51

Description

The array returned for a n-best call to classifyDocument is sorted ascendant instead of descendant.

Ex:

47-best: [ClassifierResult{category='香川県', score=32.28281232047167},
ClassifierResult{category='宮崎県', score=32.28969992600906}, ......,
ClassifierResult{category='愛知県', score=32.487981016587796},
ClassifierResult{category='東京都', score=32.49189358054859},
ClassifierResult{category='北海道', score=32.49811200756193}]

(classification of documents for Japanese prefectures)

Inside the classifyDocument method, just before the return statement we found this line:

Collections.reverse(result);

Is this a mistake or a design choice? (we are not sure, hence the "Minor" priority)