X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fanalysis%2FAAFrequency.java;h=61f3b7f3b391024ce2b5355975b143871ad60f3f;hb=80634498762666e6acc92368716bf1a4d4f42f7b;hp=a792d24bd3a086b01bb22e9f4587ca797d8a79b2;hpb=ff450fad8709ae81919af7a15ea382af7292794c;p=jalview.git
diff --git a/src/jalview/analysis/AAFrequency.java b/src/jalview/analysis/AAFrequency.java
index a792d24..61f3b7f 100755
--- a/src/jalview/analysis/AAFrequency.java
+++ b/src/jalview/analysis/AAFrequency.java
@@ -147,8 +147,8 @@ public class AAFrequency
{
if (sequences[row] == null)
{
- System.err
- .println("WARNING: Consensus skipping null sequence - possible race condition.");
+ System.err.println(
+ "WARNING: Consensus skipping null sequence - possible race condition.");
continue;
}
if (sequences[row].getLength() > column)
@@ -315,7 +315,7 @@ public class AAFrequency
// always set ranges again
gaprow.graphMax = nseq;
gaprow.graphMin = 0;
- double scale = 0.8/nseq;
+ double scale = 0.8 / nseq;
for (int i = startCol; i < endCol; i++)
{
ProfileI profile = profiles.get(i);
@@ -333,9 +333,9 @@ public class AAFrequency
String description = "" + gapped;
- gaprow.annotations[i] = new Annotation("", description,
- '\0', gapped, jalview.util.ColorUtils.bleachColour(
- Color.DARK_GRAY, (float) scale * gapped));
+ gaprow.annotations[i] = new Annotation("", description, '\0', gapped,
+ jalview.util.ColorUtils.bleachColour(Color.DARK_GRAY,
+ (float) scale * gapped));
}
}
@@ -344,7 +344,8 @@ public class AAFrequency
*
* - the full profile (percentages of all residues present), if
* showSequenceLogo is true, or
- * - just the modal (most common) residue(s), if showSequenceLogo is false
+ * - just the modal (most common) residue(s), if showSequenceLogo is
+ * false
*
* Percentages are as a fraction of all sequence, or only ungapped sequences
* if ignoreGaps is true.
@@ -365,8 +366,8 @@ public class AAFrequency
String description = null;
if (counts != null && showSequenceLogo)
{
- int normaliseBy = ignoreGaps ? profile.getNonGapped() : profile
- .getHeight();
+ int normaliseBy = ignoreGaps ? profile.getNonGapped()
+ : profile.getHeight();
description = counts.getTooltip(normaliseBy, dp);
}
else
@@ -397,7 +398,7 @@ public class AAFrequency
* contains
*
*
- * [profileType, numberOfValues, nonGapCount, charValue1, percentage1, charValue2, percentage2, ...]
+ * [profileType, numberOfValues, totalPercent, charValue1, percentage1, charValue2, percentage2, ...]
* in descending order of percentage value
*
*
@@ -410,7 +411,6 @@ public class AAFrequency
*/
public static int[] extractProfile(ProfileI profile, boolean ignoreGaps)
{
- int[] rtnval = new int[64];
ResidueCount counts = profile.getCounts();
if (counts == null)
{
@@ -421,29 +421,51 @@ public class AAFrequency
char[] symbols = symbolCounts.symbols;
int[] values = symbolCounts.values;
QuickSort.sort(values, symbols);
- int nextArrayPos = 2;
int totalPercentage = 0;
- final int divisor = ignoreGaps ? profile.getNonGapped() : profile
- .getHeight();
+ final int divisor = ignoreGaps ? profile.getNonGapped()
+ : profile.getHeight();
/*
* traverse the arrays in reverse order (highest counts first)
*/
+ int[] result = new int[3 + 2 * symbols.length];
+ int nextArrayPos = 3;
+ int nonZeroCount = 0;
+
for (int i = symbols.length - 1; i >= 0; i--)
{
int theChar = symbols[i];
int charCount = values[i];
-
- rtnval[nextArrayPos++] = theChar;
final int percentage = (charCount * 100) / divisor;
- rtnval[nextArrayPos++] = percentage;
+ if (percentage == 0)
+ {
+ /*
+ * this count (and any remaining) round down to 0% - discard
+ */
+ break;
+ }
+ nonZeroCount++;
+ result[nextArrayPos++] = theChar;
+ result[nextArrayPos++] = percentage;
totalPercentage += percentage;
}
- rtnval[0] = symbols.length;
- rtnval[1] = totalPercentage;
- int[] result = new int[rtnval.length + 1];
+
+ /*
+ * truncate array if any zero values were discarded
+ */
+ if (nonZeroCount < symbols.length)
+ {
+ int[] tmp = new int[3 + 2 * nonZeroCount];
+ System.arraycopy(result, 0, tmp, 0, tmp.length);
+ result = tmp;
+ }
+
+ /*
+ * fill in 'header' values
+ */
result[0] = AlignmentAnnotation.SEQUENCE_PROFILE;
- System.arraycopy(rtnval, 0, result, 1, rtnval.length);
+ result[1] = nonZeroCount;
+ result[2] = totalPercentage;
return result;
}
@@ -453,14 +475,15 @@ public class AAFrequency
* contains
*
*
- * [profileType, numberOfValues, totalCount, charValue1, percentage1, charValue2, percentage2, ...]
+ * [profileType, numberOfValues, totalPercentage, charValue1, percentage1, charValue2, percentage2, ...]
* in descending order of percentage value, where the character values encode codon triplets
*
*
* @param hashtable
* @return
*/
- public static int[] extractCdnaProfile(Hashtable hashtable,
+ public static int[] extractCdnaProfile(
+ Hashtable hashtable,
boolean ignoreGaps)
{
// this holds #seqs, #ungapped, and then codon count, indexed by encoded
@@ -491,9 +514,16 @@ public class AAFrequency
{
break; // nothing else of interest here
}
+ final int percentage = codonCount * 100 / divisor;
+ if (percentage == 0)
+ {
+ /*
+ * this (and any remaining) values rounded down to 0 - discard
+ */
+ break;
+ }
distinctValuesCount++;
result[j++] = codons[i];
- final int percentage = codonCount * 100 / divisor;
result[j++] = percentage;
totalPercentage += percentage;
}
@@ -517,7 +547,7 @@ public class AAFrequency
* the consensus data stores to be populated (one per column)
*/
public static void calculateCdna(AlignmentI alignment,
- Hashtable[] hconsensus)
+ Hashtable[] hconsensus)
{
final char gapCharacter = alignment.getGapCharacter();
List mappings = alignment.getCodonFrames();
@@ -530,7 +560,7 @@ public class AAFrequency
for (int col = 0; col < cols; col++)
{
// todo would prefer a Java bean for consensus data
- Hashtable columnHash = new Hashtable();
+ Hashtable columnHash = new Hashtable<>();
// #seqs, #ungapped seqs, counts indexed by (codon encoded + 1)
int[] codonCounts = new int[66];
codonCounts[0] = alignment.getSequences().size();
@@ -541,8 +571,8 @@ public class AAFrequency
{
continue;
}
- List codons = MappingUtils
- .findCodonsFor(seq, col, mappings);
+ List codons = MappingUtils.findCodonsFor(seq, col,
+ mappings);
for (char[] codon : codons)
{
int codonEncoded = CodingUtils.encodeCodon(codon);
@@ -575,7 +605,8 @@ public class AAFrequency
*/
public static void completeCdnaConsensus(
AlignmentAnnotation consensusAnnotation,
- Hashtable[] consensusData, boolean showProfileLogo, int nseqs)
+ Hashtable[] consensusData, boolean showProfileLogo,
+ int nseqs)
{
if (consensusAnnotation == null
|| consensusAnnotation.annotations == null
@@ -590,7 +621,7 @@ public class AAFrequency
consensusAnnotation.scaleColLabel = true;
for (int col = 0; col < consensusData.length; col++)
{
- Hashtable hci = consensusData[col];
+ Hashtable hci = consensusData[col];
if (hci == null)
{
// gapped protein column?
@@ -622,10 +653,10 @@ public class AAFrequency
int modalCodonEncoded = codons[codons.length - 1];
int modalCodonCount = sortedCodonCounts[codons.length - 1];
- String modalCodon = String.valueOf(CodingUtils
- .decodeCodon(modalCodonEncoded));
- if (sortedCodonCounts.length > 1
- && sortedCodonCounts[codons.length - 2] == sortedCodonCounts[codons.length - 1])
+ String modalCodon = String
+ .valueOf(CodingUtils.decodeCodon(modalCodonEncoded));
+ if (sortedCodonCounts.length > 1 && sortedCodonCounts[codons.length
+ - 2] == sortedCodonCounts[codons.length - 1])
{
/*
* two or more codons share the modal count
@@ -684,8 +715,8 @@ public class AAFrequency
{
if (samePercent.length() > 0)
{
- mouseOver.append(samePercent).append(": ")
- .append(lastPercent).append("% ");
+ mouseOver.append(samePercent).append(": ").append(lastPercent)
+ .append("% ");
}
samePercent.setLength(0);
samePercent.append(codon);