Skip to content

Commit 2d06b9b

Browse files
committed
Fix evalution issue with PredicateArgumentEvaluator.
- `evaluateSense` method requires attributes as part of the evaluation. - `evaluate` methods remotes attributes in the gold and predicted views. - Calling `evaluate` before `evaluateSense` causes the evaluation to fail as the attributes are missing now.
1 parent 73bb4e6 commit 2d06b9b

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

core-utilities/src/main/java/edu/illinois/cs/cogcomp/core/experiments/evaluators/PredicateArgumentEvaluator.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ public void evaluateSense(ClassificationTester senseTester, View goldView, View
5656
* @param tester The multi-class {@link ClassificationTester} for the argument labels
5757
*/
5858
public void evaluate(ClassificationTester tester, View goldView, View predictionView) {
59-
super.cleanAttributes(goldView, predictionView);
6059
gold = (PredicateArgumentView) goldView;
6160
prediction = (PredicateArgumentView) predictionView;
6261
goldToPredictionPredicateMapping = getGoldToPredictionPredicateMapping();

0 commit comments

Comments
 (0)