and another bugfix for merging overlaps

HeidelTime · Mar 18, 2014 · d025656 · d025656
1 parent df67acf
commit d025656
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/de/unihd/dbs/uima/annotator/heideltime/HeidelTime.java b/src/de/unihd/dbs/uima/annotator/heideltime/HeidelTime.java
@@ -1840,6 +1840,9 @@ private void deleteOverlappedTimexesPostprocessing(JCas jcas) {
 				// check timex value attribute string length
 				if(longestTimex == null) {
 					longestTimex = t;
+				} else if(longestTimex.getTimexValue().length() == t.getTimexValue().length()) {
+					if(t.getBegin() < longestTimex.getBegin())
+						longestTimex = t;
 				} else if(longestTimex.getTimexValue().length() < t.getTimexValue().length()) {
 					longestTimex = t;
 				}
@@ -1856,7 +1859,7 @@ private void deleteOverlappedTimexesPostprocessing(JCas jcas) {
 				// disassemble and remember the token ids
 				String[] tokenizedTokenIds = t.getAllTokIds().split("<-->");
 				for(Integer i = 1; i < tokenizedTokenIds.length; i++) {
-					if(!tokenIds.contains(tokenizedTokenIds[i])) {
+					if(!tokenIds.contains(Integer.parseInt(tokenizedTokenIds[i]))) {
 						tokenIds.add(Integer.parseInt(tokenizedTokenIds[i]));
 					}
 				}