JAL-2480 clarify that findOverlaps excludes spanning contact features
[jalview.git] / src / jalview / datamodel / Sequence.java
index 7ecb07a..cdfde5c 100755 (executable)
@@ -31,9 +31,11 @@ import jalview.util.StringUtils;
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.BitSet;
 import java.util.Collections;
 import java.util.Enumeration;
 import java.util.List;
+import java.util.ListIterator;
 import java.util.Vector;
 
 import com.stevesoft.pat.Regex;
@@ -42,10 +44,7 @@ import fr.orsay.lri.varna.models.rna.RNA;
 
 /**
  * 
- * Implements the SequenceI interface for a char[] based sequence object.
- * 
- * @author $author$
- * @version $Revision$
+ * Implements the SequenceI interface for a char[] based sequence object
  */
 public class Sequence extends ASequence implements SequenceI
 {
@@ -258,9 +257,8 @@ public class Sequence extends ASequence implements SequenceI
   protected void initSeqFrom(SequenceI seq,
           AlignmentAnnotation[] alAnnotation)
   {
-    char[] oseq = seq.getSequence();
-    initSeqAndName(seq.getName(), Arrays.copyOf(oseq, oseq.length),
-            seq.getStart(), seq.getEnd());
+    char[] oseq = seq.getSequence(); // returns a copy of the array
+    initSeqAndName(seq.getName(), oseq, seq.getStart(), seq.getEnd());
 
     description = seq.getDescription();
     if (seq != datasetSequence)
@@ -543,6 +541,7 @@ public class Sequence extends ASequence implements SequenceI
   {
     this.sequence = seq.toCharArray();
     checkValidRange();
+    sequenceChanged();
   }
 
   @Override
@@ -560,7 +559,9 @@ public class Sequence extends ASequence implements SequenceI
   @Override
   public char[] getSequence()
   {
-    return sequence;
+    // return sequence;
+    return sequence == null ? null : Arrays.copyOf(sequence,
+            sequence.length);
   }
 
   /*
@@ -663,50 +664,73 @@ public class Sequence extends ASequence implements SequenceI
     return this.description;
   }
 
-  /*
-   * (non-Javadoc)
-   * 
-   * @see jalview.datamodel.SequenceI#findIndex(int)
+  /**
+   * {@inheritDoc}
    */
   @Override
   public int findIndex(int pos)
   {
     /*
-     * use a valid nearby cursor if available
+     * use a valid, hopefully nearby, cursor if available
      */
-    if (cursor != null && cursor.sequence == this
-            && cursor.token == changeCount)
+    if (isValidCursor(cursor))
     {
       return findIndex(pos, cursor);
     }
 
     int j = start;
     int i = 0;
-    // Rely on end being at least as long as the length of the sequence.
+    int startColumn = 0;
+
+    /*
+     * traverse sequence from the start counting gaps; make a note of
+     * the column of the first residue to save in the cursor
+     */
     while ((i < sequence.length) && (j <= end) && (j <= pos))
     {
       if (!Comparison.isGap(sequence[i]))
       {
+        if (j == start)
+        {
+          startColumn = i;
+        }
         j++;
       }
       i++;
     }
 
-    if ((j == end) && (j < pos))
+    if (j == end && j < pos)
     {
       return end + 1;
     }
-    else
-    {
-      updateCursor(pos, i);
-      return i;
-    }
+
+    updateCursor(pos, i, startColumn);
+    return i;
   }
 
-  protected void updateCursor(int residuePos, int column)
+  /**
+   * Updates the cursor to the latest found residue and column position
+   * 
+   * @param residuePos
+   *          (start..)
+   * @param column
+   *          (1..)
+   * @param startColumn
+   *          column position of the first sequence residue
+   */
+  protected void updateCursor(int residuePos, int column, int startColumn)
   {
-    // TODO probably want to synchronize this on something
-    cursor = new SequenceCursor(this, residuePos, column, this.changeCount);
+    /*
+     * preserve end residue column provided cursor was valid
+     */
+    int endColumn = isValidCursor(cursor) ? cursor.lastResidueColumn : 0;
+    if (residuePos == this.end)
+    {
+      endColumn = column;
+    }
+
+    cursor = new SequenceCursor(this, residuePos, column, startColumn,
+            endColumn, this.changeCount);
   }
 
   /**
@@ -720,7 +744,7 @@ public class Sequence extends ASequence implements SequenceI
    */
   protected int findIndex(int pos, SequenceCursor curs)
   {
-    if (curs.sequence != this || curs.token != changeCount)
+    if (!isValidCursor(curs))
     {
       /*
        * wrong or invalidated cursor, compute de novo
@@ -736,8 +760,7 @@ public class Sequence extends ASequence implements SequenceI
     /*
      * move left or right to find pos from hint.position
      */
-    int col = curs.columnPosition - 1; // convert from base 1 to 0-based array
-                                       // index
+    int col = curs.columnPosition - 1; // convert from base 1 to base 0
     int newPos = curs.residuePosition;
     int delta = newPos > pos ? -1 : 1;
 
@@ -755,264 +778,252 @@ public class Sequence extends ASequence implements SequenceI
     }
 
     col++; // convert back to base 1
-    updateCursor(pos, col);
+    updateCursor(pos, col, curs.firstResidueColumn);
 
     return col;
   }
 
+  /**
+   * {@inheritDoc}
+   */
   @Override
-  public int findPosition(final int i)
+  public Range findPositions(int fromColumn, int toColumn)
   {
-    /*
-     * use a valid nearby cursor if available
-     */
-    if (cursor != null && cursor.sequence == this
-            && cursor.token == changeCount)
+    if (toColumn < fromColumn || fromColumn < 1)
     {
-      return findPosition(i + 1, cursor);
+      return null;
     }
 
-    int j = 0;
-    int pos = start;
-    int seqlen = sequence.length;
-    while ((j < i) && (j < seqlen))
+    /*
+     * find the first non-gapped position, if any
+     */
+    int firstPosition = 0;
+    int col = fromColumn - 1;
+    int length = sequence.length;
+    while (col < length && col < toColumn)
     {
-      if (!Comparison.isGap(sequence[j]))
+      if (!Comparison.isGap(sequence[col]))
       {
-        pos++;
+        firstPosition = findPosition(col++);
+        break;
       }
+      col++;
+    }
 
-      j++;
+    if (firstPosition == 0)
+    {
+      return null;
     }
 
-    if (j == i && !Comparison.isGap(sequence[i]))
+    /*
+     * find the last non-gapped position
+     */
+    int lastPosition = firstPosition;
+    while (col < length && col < toColumn)
     {
-      updateCursor(pos, i + 1);
+      if (!Comparison.isGap(sequence[col++]))
+      {
+        lastPosition++;
+      }
     }
 
-    return pos;
+    return new Range(firstPosition, lastPosition);
   }
 
   /**
-   * Answers the sequence position (start..) for the given aligned column
-   * position (1..), given a hint of a cursor in the neighbourhood. The cursor
-   * may lie left of, at, or to the right of the column position.
-   * 
-   * @param col
-   * @param curs
-   * @return
+   * {@inheritDoc}
    */
-  protected int findPosition(final int col, SequenceCursor curs)
+  @Override
+  public int findPosition(final int column)
   {
-    if (curs.sequence != this || curs.token != changeCount)
+    /*
+     * use a valid, hopefully nearby, cursor if available
+     */
+    if (isValidCursor(cursor))
     {
-      /*
-       * wrong or invalidated cursor, compute de novo
-       */
-      return findPosition(col - 1);// ugh back to base 0
+      return findPosition(column + 1, cursor);
     }
+    
+    // TODO recode this more naturally i.e. count residues only
+    // as they are found, not 'in anticipation'
 
-    if (curs.columnPosition == col)
+    /*
+     * traverse the sequence counting gaps; note the column position
+     * of the first residue, to save in the cursor
+     */
+    int firstResidueColumn = 0;
+    int lastPosFound = 0;
+    int lastPosFoundColumn = 0;
+    int seqlen = sequence.length;
+
+    if (seqlen > 0 && !Comparison.isGap(sequence[0]))
     {
-      return curs.residuePosition; // easy case :-)
+      lastPosFound = start;
+      lastPosFoundColumn = 0;
     }
 
-    /*
-     * move left or right to find pos from cursor position
-     */
-    int column = curs.columnPosition - 1; // to base 0
-    int newPos = curs.residuePosition;
-    int delta = curs.columnPosition > col ? -1 : 1;
-    boolean gapped = false;
+    int j = 0;
+    int pos = start;
 
-    while (column != col - 1)
+    while (j < column && j < seqlen)
     {
-      column += delta; // shift one column left or right
-      if (column < 0 || column == sequence.length)
+      if (!Comparison.isGap(sequence[j]))
       {
-        break;
+        lastPosFound = pos;
+        lastPosFoundColumn = j;
+        if (pos == this.start)
+        {
+          firstResidueColumn = j;
+        }
+        pos++;
       }
-      gapped = Comparison.isGap(sequence[column]);
-      if (!gapped)
+      j++;
+    }
+    if (j < seqlen && !Comparison.isGap(sequence[j]))
+    {
+      lastPosFound = pos;
+      lastPosFoundColumn = j;
+      if (pos == this.start)
       {
-        newPos += delta;
+        firstResidueColumn = j;
       }
     }
 
     /*
-     * hack to give position to the right if on a gap
-     * pending resolution of JAL-2562
+     * update the cursor to the last residue position found (if any)
+     * (converting column position to base 1)
      */
-    if (delta > 0 && gapped)
+    if (lastPosFound != 0)
     {
-      newPos++;
+      updateCursor(lastPosFound, lastPosFoundColumn + 1,
+              firstResidueColumn + 1);
     }
 
-    return newPos;
+    return pos;
   }
 
   /**
-   * {@inheritDoc}
+   * Answers true if the given cursor is not null, is for this sequence object,
+   * and has a token value that matches this object's changeCount, else false.
+   * This allows us to ignore a cursor as 'stale' if the sequence has been
+   * modified since the cursor was created.
+   * 
+   * @param curs
+   * @return
    */
-  @Override
-  public Range findPositions(int fromCol, int toCol)
+  protected boolean isValidCursor(SequenceCursor curs)
   {
-    if (cursor != null && cursor.sequence == this
-            && cursor.token == changeCount)
+    if (curs == null || curs.sequence != this || curs.token != changeCount)
     {
-      return findPositions(fromCol, toCol, cursor);
+      return false;
     }
-
     /*
-     * count residues before fromCol
+     * sanity check against range
      */
-    int j = 0;
-    int count = 0;
-    int seqlen = sequence.length;
-    while (j < fromCol && j < seqlen)
+    if (curs.columnPosition < 0 || curs.columnPosition > sequence.length)
     {
-      if (!Comparison.isGap(sequence[j]))
-      {
-        count++;
-      }
-      j++;
+      return false;
     }
-
-    /*
-     * find first and last residues between fromCol and toCol
-     */
-    int firstPos = 0;
-    int lastPos = 0;
-    boolean foundFirst = false;
-    
-    while (j <= toCol && j < seqlen)
+    if (curs.residuePosition < start || curs.residuePosition > end)
     {
-      if (!Comparison.isGap(sequence[j]))
-      {
-        count++;
-        if (!foundFirst)
-        {
-          firstPos = count;
-          foundFirst = true;
-        }
-        lastPos = count;
-      }
-      j++;
-    }
-
-    if (firstPos == 0)
-    {
-      /*
-       * no residues in this range
-       */
-      return null;
+      return false;
     }
-
-    /*
-     * adjust for sequence start coordinate
-     */
-    firstPos += start - 1;
-    lastPos += start - 1;
-
-    return new Range(firstPos, lastPos);
+    return true;
   }
 
   /**
-   * Returns the range of sequence positions included in the given alignment
-   * position range. If no positions are included (the range is entirely gaps),
-   * then returns null. The cursor parameter may provide a starting position in
-   * the neighbourhood of the search (which may be left of, right of, or
-   * overlapping the search region).
+   * Answers the sequence position (start..) for the given aligned column
+   * position (1..), given a hint of a cursor in the neighbourhood. The cursor
+   * may lie left of, at, or to the right of the column position.
    * 
-   * @param fromCol
-   *          start column of region (0..)
-   * @param toCol
-   *          end column of region (0..)
+   * @param col
    * @param curs
    * @return
    */
-  protected Range findPositions(int fromCol, int toCol, SequenceCursor curs)
+  protected int findPosition(final int col, SequenceCursor curs)
   {
-    if (curs.sequence != this || curs.token != changeCount)
+    if (!isValidCursor(curs))
     {
       /*
        * wrong or invalidated cursor, compute de novo
        */
-      return findPositions(fromCol, toCol);
+      return findPosition(col - 1);// ugh back to base 0
     }
 
-    /*
-     * keep this simple...first step from cursor to fromCol...
-     */
-    final int seqlen = sequence.length;
-    int resNo = curs.residuePosition;
-    int col = curs.columnPosition - 1; // from base 1 to base 0
-    if (col != fromCol)
+    if (curs.columnPosition == col)
     {
-      int delta = col > fromCol ? -1 : 1;
-      while (col != fromCol && col >= 0 && col < seqlen)
-      {
-        if (!Comparison.isGap(sequence[col]))
-        {
-          resNo += delta;
-        }
-        col += delta;
-      }
+      cursor = curs; // in case this method becomes public
+      return curs.residuePosition; // easy case :-)
     }
 
-    if (col < fromCol || col == seqlen)
+    if (curs.lastResidueColumn > 0 && curs.lastResidueColumn < col)
     {
       /*
-       * sequence lies to the left of the target region
+       * sequence lies entirely to the left of col
+       * - return last residue + 1
        */
-      return null;
+      return end + 1;
     }
 
-    /*
-     * resNo is now the residue at fromCol (if not gapped), else the one
-     * before it (if delta == 1), else the one after (if delta == -1);
-     * we want the residue before fromCol
-     */
-    if (!Comparison.isGap(sequence[fromCol]))
+    if (curs.firstResidueColumn > 0 && curs.firstResidueColumn > col)
     {
-      resNo--;
-    }
-    else if (curs.columnPosition > fromCol)
-    {
-      resNo -= 2;
+      /*
+       * sequence lies entirely to the right of col
+       * - return first residue
+       */
+      return start;
     }
 
+    // todo could choose closest to col out of column,
+    // firstColumnPosition, lastColumnPosition as a start point
+
     /*
-     * now first and last residues between fromCol and toCol
+     * move left or right to find pos from cursor position
      */
-    int firstPos = 0;
-    int lastPos = 0;
-    boolean foundFirst = false;
+    int firstResidueColumn = curs.firstResidueColumn;
+    int column = curs.columnPosition - 1; // to base 0
+    int newPos = curs.residuePosition;
+    int delta = curs.columnPosition > col ? -1 : 1;
+    boolean gapped = false;
+    int lastFoundPosition = curs.residuePosition;
+    int lastFoundPositionColumn = curs.columnPosition;
 
-    while (col <= toCol && col < seqlen)
+    while (column != col - 1)
     {
-      if (!Comparison.isGap(sequence[col]))
+      column += delta; // shift one column left or right
+      if (column < 0 || column == sequence.length)
+      {
+        break;
+      }
+      gapped = Comparison.isGap(sequence[column]);
+      if (!gapped)
       {
-        resNo++;
-        if (!foundFirst)
+        newPos += delta;
+        lastFoundPosition = newPos;
+        lastFoundPositionColumn = column + 1;
+        if (lastFoundPosition == this.start)
         {
-          firstPos = resNo;
-          foundFirst = true;
+          firstResidueColumn = column + 1;
         }
-        lastPos = resNo;
       }
-      col++;
     }
 
-    if (firstPos == 0)
+    if (cursor == null || lastFoundPosition != cursor.residuePosition)
     {
-      /*
-       * no residues in this range
-       */
-      return null;
+      updateCursor(lastFoundPosition, lastFoundPositionColumn,
+              firstResidueColumn);
     }
 
-    return new Range(firstPos, lastPos);
+    /*
+     * hack to give position to the right if on a gap
+     * or beyond the length of the sequence (see JAL-2562)
+     */
+    if (delta > 0 && (gapped || column >= sequence.length))
+    {
+      newPos++;
+    }
+
+    return newPos;
   }
 
   /**
@@ -1099,6 +1110,40 @@ public class Sequence extends ASequence implements SequenceI
   }
 
   @Override
+  public BitSet getInsertionsAsBits()
+  {
+    BitSet map = new BitSet();
+    int lastj = -1, j = 0;
+    int pos = start;
+    int seqlen = sequence.length;
+    while ((j < seqlen))
+    {
+      if (jalview.util.Comparison.isGap(sequence[j]))
+      {
+        if (lastj == -1)
+        {
+          lastj = j;
+        }
+      }
+      else
+      {
+        if (lastj != -1)
+        {
+          map.set(lastj, j);
+          lastj = -1;
+        }
+      }
+      j++;
+    }
+    if (lastj != -1)
+    {
+      map.set(lastj, j);
+      lastj = -1;
+    }
+    return map;
+  }
+
+  @Override
   public void deleteChars(int i, int j)
   {
     int newstart = start, newend = end;
@@ -1113,9 +1158,9 @@ public class Sequence extends ASequence implements SequenceI
     // the very large sequence case
     int eindex = -1, sindex = -1;
     boolean ecalc = false, scalc = false;
-    for (int s = i; s < j; s++)
+    for (int s = i; s < j && s < sequence.length; s++)
     {
-      if (jalview.schemes.ResidueProperties.aaIndex[sequence[s]] != 23)
+      if (!Comparison.isGap(sequence[s]))
       {
         if (createNewDs)
         {
@@ -1171,6 +1216,7 @@ public class Sequence extends ASequence implements SequenceI
     start = newstart;
     end = newend;
     sequence = tmp;
+    sequenceChanged();
   }
 
   @Override
@@ -1201,6 +1247,7 @@ public class Sequence extends ASequence implements SequenceI
     }
 
     sequence = tmp;
+    sequenceChanged();
   }
 
   @Override
@@ -1395,7 +1442,7 @@ public class Sequence extends ASequence implements SequenceI
 
   private boolean _isNa;
 
-  private long _seqhash = 0;
+  private int _seqhash = 0;
 
   /**
    * Answers false if the sequence is more than 85% nucleotide (ACGTU), else
@@ -1759,14 +1806,46 @@ public class Sequence extends ASequence implements SequenceI
    * {@inheritDoc}
    */
   @Override
-  public List<SequenceFeature> findFeatures(int from, int to,
+  public List<SequenceFeature> findFeatures(int fromColumn, int toColumn,
           String... types)
   {
+    int startPos = findPosition(fromColumn - 1); // convert base 1 to base 0
+    int endPos = findPosition(toColumn - 1);
+
+    List<SequenceFeature> result = new ArrayList<>();
     if (datasetSequence != null)
     {
-      return datasetSequence.findFeatures(from, to, types);
+      result = datasetSequence.getFeatures().findFeatures(startPos, endPos,
+              types);
+    }
+    else
+    {
+      result = sequenceFeatureStore.findFeatures(startPos, endPos, types);
+    }
+
+    /*
+     * if the start or end column is gapped, startPos or endPos may be to the 
+     * left or right, and we may have included adjacent or enclosing features;
+     * remove any that are not enclosing features
+     */
+    if (endPos > this.end || Comparison.isGap(sequence[fromColumn - 1])
+            || Comparison.isGap(sequence[toColumn - 1]))
+    {
+      ListIterator<SequenceFeature> it = result.listIterator();
+      while (it.hasNext())
+      {
+        SequenceFeature sf = it.next();
+        int featureStartColumn = findIndex(sf.getBegin());
+        int featureEndColumn = findIndex(sf.getEnd());
+        if (featureStartColumn > toColumn
+                        || featureEndColumn < fromColumn)
+        {
+          it.remove();
+        }
+      }
     }
-    return sequenceFeatureStore.findFeatures(from, to, types);
+
+    return result;
   }
 
   /**
@@ -1774,8 +1853,38 @@ public class Sequence extends ASequence implements SequenceI
    * token that has to match the one presented by the cursor
    */
   @Override
-  public void zapCursor()
+  public void sequenceChanged()
   {
     changeCount++;
   }
+
+  /**
+   * {@inheritDoc}
+   */
+  @Override
+  public int replace(char c1, char c2)
+  {
+    if (c1 == c2)
+    {
+      return 0;
+    }
+    int count = 0;
+    synchronized (sequence)
+    {
+      for (int c = 0; c < sequence.length; c++)
+      {
+        if (sequence[c] == c1)
+        {
+          sequence[c] = c2;
+          count++;
+        }
+      }
+    }
+    if (count > 0)
+    {
+      sequenceChanged();
+    }
+
+    return count;
+  }
 }