X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Futil%2FComparison.java;h=cd98ee74bf560bf426c948950cc1e094809ff3a4;hb=05688ef13325c699644487ce1c08a53927f0eeb0;hp=286bfb257332dac8e1f2481947c616cd6d4a99e6;hpb=4728b1bd60655ad9f269438e505e70f23c9c83f3;p=jalview.git

diff --git a/src/jalview/util/Comparison.java b/src/jalview/util/Comparison.java
index 286bfb2..cd98ee7 100644
--- a/src/jalview/util/Comparison.java
+++ b/src/jalview/util/Comparison.java
@@ -261,40 +261,46 @@ public class Comparison
 
   /**
    * Overloaded method signature to test whether a single sequence is nucleotide
-   * (that is, more than 85% CGTA)
+   * (that is, more than 85% CGTAUNX)
    * 
    * @param seq
    * @return
    */
   public static final boolean isNucleotide(SequenceI seq)
   {
-    if (seq==null)
+    if (seq == null)
     {
       return false;
     }
     long ntCount = 0;
     long aaCount = 0;
+    long nCount = 0;
 
     int len = seq.getLength();
     for (int i = 0; i < len; i++)
     {
       char c = seq.getCharAt(i);
-      if (isNucleotide(c))
+      if (isNucleotide(c) || isX(c))
       {
         ntCount++;
       }
       else if (!isGap(c))
       {
         aaCount++;
+        if (isN(c))
+        {
+          nCount++;
+        }
       }
     }
     /*
      * Check for nucleotide count > 85% of total count (in a form that evades
      * int / float conversion or divide by zero).
      */
-    if (ntCount * 100 > EIGHTY_FIVE * (ntCount + aaCount))
+    if ((ntCount + nCount) * 100 > EIGHTY_FIVE * (ntCount + aaCount))
     {
-      return true;
+      return ntCount > 0; // all N is considered protein. Could use a threshold
+                          // here too
     }
     else
     {
@@ -317,17 +323,18 @@ public class Comparison
       return false;
     }
     // true if we have seen a nucleotide sequence
-    boolean na=false;
+    boolean na = false;
     for (SequenceI seq : seqs)
     {
       if (seq == null)
       {
         continue;
       }
-      na=true;
+      na = true;
       // TODO could possibly make an informed guess just from the first sequence
       // to save a lengthy calculation
-      if (seq.isProtein()) {
+      if (seq.isProtein())
+      {
         // if even one looks like protein, the alignment is protein
         return false;
       }
@@ -347,7 +354,6 @@ public class Comparison
     {
       c -= TO_UPPER_CASE;
     }
-
     switch (c)
     {
     case 'A':
@@ -360,6 +366,28 @@ public class Comparison
     return false;
   }
 
+  public static boolean isN(char c)
+  {
+    switch (c)
+    {
+    case 'N':
+    case 'n':
+      return true;
+    }
+    return false;
+  }
+
+  public static boolean isX(char c)
+  {
+    switch (c)
+    {
+    case 'X':
+    case 'x':
+      return true;
+    }
+    return false;
+  }
+
   /**
    * Answers true if every character in the string is one of aAcCgGtTuU, or
    * (optionally) a gap character (dot, dash, space), else false