You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ctakes.apache.org by tm...@apache.org on 2013/08/05 22:29:38 UTC
svn commit: r1510710 -
/ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java
Author: tmill
Date: Mon Aug 5 20:29:38 2013
New Revision: 1510710
URL: http://svn.apache.org/r1510710
Log:
Cleanup of commented-out code.
Modified:
ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java
Modified: ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java
URL: http://svn.apache.org/viewvc/ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java?rev=1510710&r1=1510709&r2=1510710&view=diff
==============================================================================
--- ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java (original)
+++ ctakes/trunk/ctakes-temporal/src/main/java/org/apache/ctakes/temporal/ae/CRFTimeAnnotator.java Mon Aug 5 20:29:38 2013
@@ -127,16 +127,6 @@ public class CRFTimeAnnotator extends Te
for (CleartkExtractor extractor : this.contextFeatureExtractors) {
features.addAll(extractor.extractWithin(jCas, token, sentence));
}
- // features from previous classifications
- // not sure this makes sense in CRF context -- linear chain CRF uses this automatically?
- // TODO - fix or delete
-// int nPreviousClassifications = 2;
-// for (int i = nPreviousClassifications; i > 0; --i) {
-// int index = tokenIndex - i;
-// String previousOutcome = index < 0 ? "O" : outcomes.get(index);
-// features.add(new Feature("PreviousOutcome_" + i, previousOutcome));
-// }
-
// features from dominating parse tree
// TODO- think abouot how to incorporate this - fix or delete
// BaseToken startToken = token;