X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FPCAPanel.java;h=133c0ce022a2c5818c5ec95ddc9768f2ff8f234f;hb=84684213f7f7c7b31d3c33dc49ca3b377a295438;hp=d1216b92a20ddd3373432074696659cf2c156389;hpb=b2f9a8d7bce642ff4011bc6d49e02bb0569fbb11;p=jalview.git

diff --git a/src/jalview/gui/PCAPanel.java b/src/jalview/gui/PCAPanel.java
index d1216b9..133c0ce 100644
--- a/src/jalview/gui/PCAPanel.java
+++ b/src/jalview/gui/PCAPanel.java
@@ -116,26 +116,34 @@ public class PCAPanel extends GPCAPanel implements Runnable,
     scoreMatrixMenu.removeAll();
     for (final String sm:ResidueProperties.scoreMatrices.keySet())
     {
-      JCheckBoxMenuItem jm=new JCheckBoxMenuItem();
-      jm.setText(MessageManager.getStringOrReturn("label.score_model", sm));
-      jm.setSelected(pcaModel.getScore_matrix().equals(sm));
-      if (pcaModel.isNucleotide()==ResidueProperties.scoreMatrices.get(sm).isDNA())
+      if (ResidueProperties.getScoreMatrix(sm) != null)
       {
-        final PCAPanel us = this;
-        jm.addActionListener(new ActionListener()
+        // create an entry for this score matrix for use in PCA
+        JCheckBoxMenuItem jm = new JCheckBoxMenuItem();
+        jm.setText(MessageManager
+                .getStringOrReturn("label.score_model", sm));
+        jm.setSelected(pcaModel.getScore_matrix().equals(sm));
+        if ((ResidueProperties.scoreMatrices.get(sm).isDNA() && ResidueProperties.scoreMatrices
+                .get(sm).isProtein())
+                || pcaModel.isNucleotide() == ResidueProperties.scoreMatrices
+                        .get(sm).isDNA())
         {
-          @Override
-          public void actionPerformed(ActionEvent e)
+          final PCAPanel us = this;
+          jm.addActionListener(new ActionListener()
           {
-            if (!pcaModel.getScore_matrix().equals((String) sm))
+            @Override
+            public void actionPerformed(ActionEvent e)
             {
-              pcaModel.setScore_matrix((String) sm);
-              Thread worker = new Thread(us);
-              worker.start();
+              if (!pcaModel.getScore_matrix().equals((String) sm))
+              {
+                pcaModel.setScore_matrix((String) sm);
+                Thread worker = new Thread(us);
+                worker.start();
+              }
             }
-          }
-        });
-        scoreMatrixMenu.add(jm);
+          });
+          scoreMatrixMenu.add(jm);
+        }
       }
     }
   }