Merge branch 'develop' into update_212_Dec_merge_with_21125_chamges

[jalview.git] / src / jalview / hmmer / HmmerCommand.java

diff --git a/src/jalview/hmmer/HmmerCommand.java b/src/jalview/hmmer/HmmerCommand.java
index f38be02..9db0ae1 100644 (file)
--- a/src/jalview/hmmer/HmmerCommand.java
+++ b/src/jalview/hmmer/HmmerCommand.java
@@ -1,7 +1,9 @@
 package jalview.hmmer;
 
 import jalview.analysis.SeqsetUtils;
+import jalview.analysis.SeqsetUtils.SequenceInfo;
 import jalview.bin.Cache;
+import jalview.bin.Console;
 import jalview.datamodel.Alignment;
 import jalview.datamodel.AlignmentAnnotation;
 import jalview.datamodel.AlignmentI;
@@ -30,6 +32,7 @@ import java.nio.file.Paths;
 import java.util.ArrayList;
 import java.util.Hashtable;
 import java.util.List;
+import java.util.Map;
 
 /**
  * Base class for hmmbuild, hmmalign and hmmsearch
@@ -60,6 +63,8 @@ public abstract class HmmerCommand implements Runnable
 
   static final String NUMBER_OF_RESULTS_KEY = "label.number_of_results";
 
+  static final String NUMBER_OF_ITERATIONS = "label.number_of_iterations";
+
   static final String TRIM_TERMINI_KEY = "label.trim_termini";
 
   static final String RETURN_N_NEW_SEQ = "label.check_for_new_sequences";
@@ -125,7 +130,7 @@ public abstract class HmmerCommand implements Runnable
    * 
    * @param seqs
    */
-  protected Hashtable stashSequences(SequenceI[] seqs)
+  protected Map<String, SequenceInfo> stashSequences(SequenceI[] seqs)
   {
     return SeqsetUtils.uniquify(seqs, true);
   }
@@ -133,12 +138,12 @@ public abstract class HmmerCommand implements Runnable
   /**
    * Restores the sequence data lost by uniquifying
    * 
-   * @param hashtable
+   * @param sequencesHash
    * @param seqs
    */
-  protected void recoverSequences(Hashtable hashtable, SequenceI[] seqs)
+  protected void recoverSequences(Map<String, SequenceInfo> sequencesHash, SequenceI[] seqs)
   {
-    SeqsetUtils.deuniquify(hashtable, seqs);
+    SeqsetUtils.deuniquify(sequencesHash, seqs);
   }
 
   /**
@@ -152,14 +157,14 @@ public abstract class HmmerCommand implements Runnable
   public boolean runCommand(List<String> commands)
           throws IOException
   {
-    List<String> args = Platform.isWindows() ? wrapWithCygwin(commands)
+    List<String> args = Platform.isWindowsAndNotJS() ? wrapWithCygwin(commands)
             : commands;
 
     try
     {
       ProcessBuilder pb = new ProcessBuilder(args);
       pb.redirectErrorStream(true); // merge syserr to sysout
-      if (Platform.isWindows())
+      if (Platform.isWindowsAndNotJS())
       {
         String path = pb.environment().get("Path");
         path = jalview.bin.Cache.getProperty("CYGWIN_PATH") + ";" + path;
@@ -192,8 +197,8 @@ public abstract class HmmerCommand implements Runnable
       int exitValue = p.exitValue();
       if (exitValue != 0)
       {
-        Cache.log.error("Command failed, return code = " + exitValue);
-        Cache.log.error("Command/args were: " + args.toString());
+        Console.error("Command failed, return code = " + exitValue);
+        Console.error("Command/args were: " + args.toString());
       }
       return exitValue == 0; // 0 is success, by convention
     } catch (Exception e)
@@ -216,7 +221,7 @@ public abstract class HmmerCommand implements Runnable
             Cache.getProperty(Preferences.CYGWIN_PATH));
     if (bash == null)
     {
-      Cache.log.error("Cygwin shell not found");
+      Console.error("Cygwin shell not found");
       return commands;
     }
 
@@ -434,7 +439,7 @@ public abstract class HmmerCommand implements Runnable
   protected String getFilePath(File resultFile, boolean isInCygwin)
   {
     String path = resultFile.getAbsolutePath();
-    if (Platform.isWindows() && isInCygwin)
+    if (Platform.isWindowsAndNotJS() && isInCygwin)
     {
       // the first backslash escapes '\' for the regular expression argument
       path = path.replaceAll("\\" + File.separator, "/");