Changeset 1888 for trunk/autoquest-core-tasktrees/src/main/java/de

Timestamp:

03/05/15 11:38:23 (10 years ago)

Author:

pharms

Message:

• corrected selection of which task to detect first.

File:

• trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/temporalrelation/SequenceForTaskDetectionRule.java (modified) (3 diffs)

trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/temporalrelation/SequenceForTaskDetectionRule.java

@@ -479 +479 @@
                     // Trie must be recreated with a longer sequence length to be sure that
                     // the found sequences occurring most often are found in their whole length
-                    appData.setTrainedSubsequenceLength(appData.getTrainedSubsequenceLength() + 1);
+                    appData.setTrainedSubsequenceLength(appData.getTrainedSubsequenceLength() + 3);
                     createNewTrie = true;
                     break;
@@ -835 +835 @@
                 // checking now, if there is one sequence, having the lowest index for any of
                 // its collisions and preserve it. If there are several ones with the same minimum
-                // index, throw an exception
+                // index, check if they are indpendent. If not, throw an exception
                 int minPosInAnySequence = Integer.MAX_VALUE;
-                InterleavingSubsequence subsequenceWithMinPos = null;
+                List<InterleavingSubsequence> subsequencesWithMinPos = new LinkedList<>();
                 
                 for (InterleavingSubsequence interleaving : interleavings) {
@@ -843 +843 @@
                         if (minPosInAnySequence > collision.getLocation().getIndex()) {
                             minPosInAnySequence = collision.getLocation().getIndex();
-                            subsequenceWithMinPos = interleaving;
+                            subsequencesWithMinPos.clear();
                         }
-                        else if (minPosInAnySequence == collision.getLocation().getIndex()) {
+                        
+                        if (minPosInAnySequence == collision.getLocation().getIndex()) {
                             // several have the same min pos --> undecidable which to prefer
-                            subsequenceWithMinPos = null;
+                            subsequencesWithMinPos.add(interleaving);
                         }
                     }
                 }
                 
-                if (subsequenceWithMinPos != null) {
+                if (subsequencesWithMinPos.size() > 0) {
                     List<Subsequence> subsequencesToRemove = new LinkedList<Subsequence>();
                     
                     for (Subsequence candidate : appData.getLastFoundSubsequences()) {
-                        if (candidate != subsequenceWithMinPos.getSubsequence()) {
+                        boolean found = false;
+                        
+                        for (InterleavingSubsequence subsequenceWithMinPos : subsequencesWithMinPos)
+                        {
+                            if (candidate == subsequenceWithMinPos.getSubsequence()) {
+                                found = true;
+                            }
+                        }
+                        
+                        if (!found) {
                             subsequencesToRemove.add(candidate);
                         }
                     }
                     
-                    for (Subsequence candidate : subsequencesToRemove) {
-                        appData.getLastFoundSubsequences().remove(candidate);
-                    }
+                    if (subsequencesToRemove.size() > 0) {
+                        for (Subsequence candidate : subsequencesToRemove) {
+                            appData.getLastFoundSubsequences().remove(candidate);
+                        }
                     
-                    continue;
+                        continue;
+                    }
                 }