From e74abe7cbec2dfc7f45ec1e02a1d36bcde63fcfc Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Tue, 26 Sep 2023 09:13:47 +0200
Subject: [PATCH 1/9] fixes typo

---
 src/main/java/qupath/ext/biop/cellpose/CellposeExtension.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/CellposeExtension.java b/src/main/java/qupath/ext/biop/cellpose/CellposeExtension.java
index 548a3da..dada0f1 100644
--- a/src/main/java/qupath/ext/biop/cellpose/CellposeExtension.java
+++ b/src/main/java/qupath/ext/biop/cellpose/CellposeExtension.java
@@ -103,6 +103,6 @@ private static void openScript(QuPathGUI qupath, String script) {
             logger.error("No script editor is available!");
             return;
         }
-        qupath.getScriptEditor().showScript("StarDist detection", script);
+        qupath.getScriptEditor().showScript("Cellpose detection", script);
     }
 }
\ No newline at end of file

From 687a5beff486d1a6be8bbe5e6117ac31001962f7 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Tue, 26 Sep 2023 09:14:18 +0200
Subject: [PATCH 2/9] adds troubleshooting in case of image pair writer error

---
 src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index cdde6ba..2e918d6 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -1198,6 +1198,7 @@ private void saveImagePairs(List<PathObject> annotations, String imageName, Imag
 
             } catch (IOException ex) {
                 logger.error(ex.getMessage());
+                logger.error("Troubleshooting:\n - Check that the channel names are correct in the builder.");
             }
         });
     }

From 6dfd3dc8c1e3395130444776cd01a0384374c467 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:25:47 +0200
Subject: [PATCH 3/9] bumps version

---
 build.gradle | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build.gradle b/build.gradle
index 8ec5ea3..5910529 100644
--- a/build.gradle
+++ b/build.gradle
@@ -25,7 +25,7 @@ ext.qupathVersion = gradle.ext.qupathVersion
 
 description = 'QuPath extension to use Cellpose'
 
-version = "0.9.2"
+version = "0.9.3-SNAPSHOT"
 
 dependencies {
     implementation "io.github.qupath:qupath-gui-fx:${qupathVersion}"

From d6f8561fa2983cbeccef250c9332bceabda8d61f Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:32:08 +0200
Subject: [PATCH 4/9] adds option to deactivate GPU in builder

---
 .../java/qupath/ext/biop/cellpose/Cellpose2D.java  |  3 ++-
 .../qupath/ext/biop/cellpose/CellposeBuilder.java  | 14 ++++++++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index 641143e..120f261 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -109,6 +109,7 @@ public class Cellpose2D {
     private final static Logger logger = LoggerFactory.getLogger(Cellpose2D.class);
 
     public ImageOp extendChannelOp;
+    public boolean useGPU;
 
     protected double simplifyDistance = 1.4;
 
@@ -746,7 +747,7 @@ private LinkedHashMap<File, TileFile> runCellpose(LinkedHashMap<File, TileFile>
 
         cellposeArguments.add("--no_npy");
 
-        cellposeArguments.add("--use_gpu");
+        if( this.useGPU ) cellposeArguments.add("--use_gpu");
 
         cellposeArguments.add("--verbose");
 
diff --git a/src/main/java/qupath/ext/biop/cellpose/CellposeBuilder.java b/src/main/java/qupath/ext/biop/cellpose/CellposeBuilder.java
index c2d5533..0b3ab3f 100644
--- a/src/main/java/qupath/ext/biop/cellpose/CellposeBuilder.java
+++ b/src/main/java/qupath/ext/biop/cellpose/CellposeBuilder.java
@@ -95,6 +95,7 @@ public class CellposeBuilder {
     private ImageOp extendChannelOp = null;
 
     private boolean doReadResultsAsynchronously = false;
+    private boolean useGPU = true;
 
     /**
      * can create a cellpose builder from a serialized JSON version of this builder.
@@ -134,9 +135,20 @@ protected CellposeBuilder(String modelPath) {
 
     }
 
+    /**
+     * overwrite use GPU
+     * @param useGPU add or remove the option
+     * @return this builder
+     */
+    public CellposeBuilder useGPU( boolean useGPU ) {
+        this.useGPU = useGPU;
+
+        return this;
+    }
 
     /**
      * Specify the training directory
+     *
      */
     public CellposeBuilder groundTruthDirectory(File groundTruthDirectory) {
         this.groundTruthDirectory = groundTruthDirectory;
@@ -771,6 +783,8 @@ public Cellpose2D build() {
         // Give it the number of threads to use
         cellpose.nThreads = nThreads;
 
+        cellpose.useGPU = useGPU;
+
         // Check the model. If it is a file, then it is a custom model
         File file = new File(this.modelNameOrPath);
         if (file.exists()) {

From f23a2b9360429b4d5a91b98623434047ae3c9da1 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:35:01 +0200
Subject: [PATCH 5/9] adds option to explicitly wait for runner to finish

---
 .../java/qupath/ext/biop/cellpose/Cellpose2D.java   | 10 ++++------
 .../ext/biop/cmd/VirtualEnvironmentRunner.java      | 13 +++++++++++--
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index 120f261..720ce73 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -754,7 +754,7 @@ private LinkedHashMap<File, TileFile> runCellpose(LinkedHashMap<File, TileFile>
         veRunner.setArguments(cellposeArguments);
 
         // Finally, we can run Cellpose
-        veRunner.runCommand();
+        veRunner.runCommand(false);
 
         return processCellposeFiles(veRunner, allTiles);
 
@@ -926,10 +926,7 @@ private void runTraining() throws IOException, InterruptedException {
         veRunner.setArguments(cellposeArguments);
 
         // Finally, we can run Cellpose
-        veRunner.runCommand();
-
-        // Wait for the process to finish
-        veRunner.getProcess().waitFor();
+        veRunner.runCommand(true);
 
         // Get the log
         this.theLog = veRunner.getProcessLog();
@@ -990,7 +987,8 @@ private ResultsTable runCellposeQC() throws IOException, InterruptedException {
 
         qcRunner.setArguments(qcArguments);
 
-        qcRunner.runCommand();
+        qcRunner.runCommand(true);
+
 
         // The results are stored in the validation directory, open them as a results table
         File qcResults = new File( getValidationDirectory(), "QC-Results" + File.separator + "Quality_Control for " + this.modelFile.getName() + ".csv");
diff --git a/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java b/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
index 6c56836..ece4ae1 100644
--- a/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
+++ b/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
@@ -123,10 +123,10 @@ public void setArguments(List<String> arguments) {
 
     /**
      * This builds, runs the command and outputs it to the logger as it is being run
-     *
+     * @param wait whether to wait for the process to end or not before exiting this command
      * @throws IOException          // In case there is an issue starting the process
      */
-    public void runCommand() throws IOException {
+    public void runCommand(boolean waitUntilDone) throws IOException {
 
         // Get how to start the command, based on the VENV Type
         List<String> command = getActivationCommand();
@@ -207,6 +207,15 @@ public void run() {
 
 
         logger.info("Virtual Environment Runner Started");
+
+        // If we ask to wait, let's wait directly here rather than handle it outside
+        if(waitUntilDone) {
+            try {
+                this.process.waitFor();
+            } catch (InterruptedException e) {
+                logger.error(e.getMessage());
+            }
+        }
     }
 
     public Process getProcess() {

From f1ffa118d160b185aefee64e343a60a86a692125 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:38:11 +0200
Subject: [PATCH 6/9] tries to do better logging in case we have no objects to
 recover

---
 .../java/qupath/ext/biop/cellpose/Cellpose2D.java  | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index 720ce73..cc13639 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -413,6 +413,11 @@ public void detectObjectsImpl(ImageData<BufferedImage> imageData, Collection<? e
 
         logger.info("All tiles and objects read, now resolving overlaps");
 
+        // In case alltiles is null, we are basically done
+        if( allTiles == null ) {
+            logger.info("No results from Cellpose", "There is nothing to recover from cellpose");
+        }
+
         // Group the candidates per parent object, as this is needed to optimize when checking for overlap
         Map<PathObject, List<CandidateObject>> candidatesPerParent = allTiles.values().stream()
                 .flatMap(t -> t.getCandidates().stream())
@@ -762,10 +767,17 @@ private LinkedHashMap<File, TileFile> runCellpose(LinkedHashMap<File, TileFile>
 
     private LinkedHashMap<File, TileFile> processCellposeFiles(VirtualEnvironmentRunner veRunner, LinkedHashMap<File, TileFile> allTiles) throws CancellationException, InterruptedException, IOException {
 
+        // Make sure that allTiles is not null, if it is, just return null
+        // as we are likely just running validation and thus do not need to give any results back
+        if (allTiles == null ) {
+            veRunner.getProcess().waitFor();
+            return null;
+        }
+
         // Build a thread pool to process reading the images in parallel
         ExecutorService executor = Executors.newFixedThreadPool(5);
 
-        if (!this.doReadResultsAsynchronously || allTiles == null) {
+        if (!this.doReadResultsAsynchronously) {
             // We need to wait for the process to finish
             veRunner.getProcess().waitFor();
             allTiles.entrySet().forEach(entry -> {

From 29fc323d769253a5ffd8cf2a40039263ed42e1f2 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:40:28 +0200
Subject: [PATCH 7/9] fixes log parsing during training

adapts regex string to match new cellpose version and
also omnipose's log
---
 .../qupath/ext/biop/cellpose/Cellpose2D.java  | 54 ++++++++++++++-----
 1 file changed, 40 insertions(+), 14 deletions(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index cc13639..01c1941 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -1058,20 +1058,25 @@ private ResultsTable parseTrainingResults() {
 
         if (this.theLog != null) {
             // Try to parse the output of Cellpose to give meaningful information to the user. This is very old school
-            // Look for "Epoch 0, Time  2.3s, Loss 1.0758, Loss Test 0.6007, LR 0.2000"
-            String epochPattern = ".*Epoch\\s*(\\d+),\\s*Time\\s*(\\d+\\.\\d)s,\\s*Loss\\s*(\\d+\\.\\d+),\\s*Loss Test\\s*(\\d+\\.\\d+),\\s*LR\\s*(\\d+\\.\\d+).*";
-            // Build Matcher
-            Pattern pattern = Pattern.compile(epochPattern);
-            Matcher m;
             for (String line : this.theLog) {
-                m = pattern.matcher(line);
-                if (m.find()) {
-                    trainingResults.incrementCounter();
-                    trainingResults.addValue("Epoch", Double.parseDouble(m.group(1)));
-                    trainingResults.addValue("Time[s]", Double.parseDouble(m.group(2)));
-                    trainingResults.addValue("Loss", Double.parseDouble(m.group(3)));
-                    trainingResults.addValue("Loss Test", Double.parseDouble(m.group(4)));
-                    trainingResults.addValue("LR", Double.parseDouble(m.group(5)));
+                Matcher m;
+                for (LogParser parser : LogParser.values()) {
+                    m = parser.getPattern().matcher(line);
+                    if (m.find()) {
+                        trainingResults.incrementCounter();
+                        trainingResults.addValue("Epoch", Double.parseDouble(m.group("epoch")));
+                        trainingResults.addValue("Time", Double.parseDouble(m.group("time")));
+                        trainingResults.addValue("Loss", Double.parseDouble(m.group("loss")));
+                        if (parser != LogParser.OMNI) { // Omnipose does not provide validation loss
+                            trainingResults.addValue("Validation Loss", Double.parseDouble(m.group("val")));
+                            trainingResults.addValue("LR", Double.parseDouble(m.group("lr")));
+
+                        } else {
+                            trainingResults.addValue("Validation Loss", Double.NaN);
+                            trainingResults.addValue("LR", Double.NaN);
+
+                        }
+                    }
                 }
             }
         }
@@ -1115,7 +1120,7 @@ public void showTrainingGraph(boolean show, boolean save) {
         //populating the series with data
         for (int i = 0; i < output.getCounter(); i++) {
             loss.getData().add(new XYChart.Data<>(output.getValue("Epoch", i), output.getValue("Loss", i)));
-            lossTest.getData().add(new XYChart.Data<>(output.getValue("Epoch", i), output.getValue("Loss Test", i)));
+            lossTest.getData().add(new XYChart.Data<>(output.getValue("Epoch", i), output.getValue("Validation Loss", i)));
 
         }
         lineChart.getData().add(loss);
@@ -1435,4 +1440,25 @@ private static class CandidateObject {
             geometry = geometry.getGeometryN(index);
         }
     }
+    public enum LogParser {
+
+        // Cellpose 2 pattern when training : "Look for "Epoch 0, Time  2.3s, Loss 1.0758, Loss Test 0.6007, LR 0.2000"
+        // Cellpose 3 pattern when training : "5, train_loss=2.6546, test_loss=2.0054, LR=0.1111, time 2.56s"
+        // Omnipose pattern when training   : "Train epoch: 10 | Time: 0.22min | last epoch: 0.74s | <sec/epoch>: 0.73s | <sec/batch>: 0.33s | <Batch Loss>: 5.076259 | <Epoch Loss>: 4.429341"
+        // WARNING: Currently Omnipose does not provide any output to the validation loss (Test loss in Cellpose)
+        CP2("Cellpose v2", ".*Epoch\\s*(?<epoch>\\d+),\\s*Time\\s*(?<time>\\d+\\.\\d)s,\\s*Loss\\s*(?<loss>\\d+\\.\\d+),\\s*Loss Test\\s*(?<val>\\d+\\.\\d+),\\s*LR\\s*(?<lr>\\d+\\.\\d+).*"),
+        CP3( "Cellpose v3", ".* (?<epoch>\\d+), train_loss=(?<loss>\\d+\\.\\d+), test_loss=(?<val>\\d+\\.\\d+), LR=(?<lr>\\d+\\.\\d+), time (?<time>\\d+\\.\\d+)s.*"),
+        OMNI("Omnipose", ".*Train epoch: (?<epoch>\\d+) \\| Time: (?<time>\\d+\\.\\d+)min .*\\<Epoch Loss\\>: (?<loss>\\d+\\.\\d+).*");
+
+        private final String name;
+        private final Pattern pattern;
+
+        LogParser(String name, String regex) {
+            this.name = name;
+            this.pattern = Pattern.compile(regex);
+        }
+
+        public String getName() {return this.name;}
+        public Pattern getPattern() {return this.pattern;}
+    }
 }
\ No newline at end of file

From 4355e4ed29572b42e226e1e2f23b890d84f410c6 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Fri, 12 Apr 2024 12:41:24 +0200
Subject: [PATCH 8/9] allows for up scaling to occur when saving training
 images

---
 src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index 01c1941..5cb402a 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -1182,18 +1182,18 @@ private void saveImagePairs(List<PathObject> annotations, String imageName, Imag
         if (annotations.isEmpty()) {
             return;
         }
-        int downsample = 1;
+        double downsample;
         if (Double.isFinite(pixelSize) && pixelSize > 0) {
-            downsample = (int) Math.round(pixelSize / originalServer.getPixelCalibration().getAveragedPixelSize().doubleValue());
+            downsample = pixelSize / originalServer.getPixelCalibration().getAveragedPixelSize().doubleValue();
+        } else {
+            downsample = 1.0;
         }
-
         AtomicInteger idx = new AtomicInteger();
-        int finalDownsample = downsample;
 
         annotations.forEach(a -> {
             int i = idx.getAndIncrement();
 
-            RegionRequest request = RegionRequest.createInstance(originalServer.getPath(), finalDownsample, a.getROI());
+            RegionRequest request = RegionRequest.createInstance(originalServer.getPath(), downsample, a.getROI());
             File imageFile = new File(saveDirectory, imageName + "_region_" + i + ".tif");
             File maskFile = new File(saveDirectory, imageName + "_region_" + i + "_masks.tif");
             try {

From 22e28118d7352fb914830dff74b64d6ec938f825 Mon Sep 17 00:00:00 2001
From: oburri <olivier.burri@gmail.com>
Date: Tue, 30 Apr 2024 08:53:27 +0200
Subject: [PATCH 9/9] adds bugfixes

---
 .../qupath/ext/biop/cellpose/Cellpose2D.java  | 14 ++++++++-----
 .../biop/cmd/VirtualEnvironmentRunner.java    | 20 ++++++++++---------
 2 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
index eb216dd..a7cf65a 100644
--- a/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
+++ b/src/main/java/qupath/ext/biop/cellpose/Cellpose2D.java
@@ -727,12 +727,15 @@ private VirtualEnvironmentRunner getVirtualEnvironmentRunner() {
      */
     private LinkedHashMap<File, TileFile> runCellpose(LinkedHashMap<File, TileFile> allTiles) throws InterruptedException, IOException {
 
-
+        // Need to define the name of the command we are running. We used to be able to use 'cellpose' for both but not since Cellpose v2
         String runCommand = this.parameters.containsKey("omni") ? "omnipose" : "cellpose";
         VirtualEnvironmentRunner veRunner = getVirtualEnvironmentRunner();
 
         // This is the list of commands after the 'python' call
-        List<String> cellposeArguments = new ArrayList<>(Arrays.asList("-W", "ignore", "-m", runCommand));
+        // We want to ignore all warnings to make sure the log is clean (-W ignore)
+        // We want to be able to call the module by name (-m)
+        // We want to make sure UTF8 mode is by default (-X utf8)
+        List<String> cellposeArguments = new ArrayList<>(Arrays.asList("-Xutf8", "-W", "ignore", "-m", runCommand));
 
         cellposeArguments.add("--dir");
         cellposeArguments.add("" + this.tempDirectory);
@@ -906,7 +909,7 @@ private void runTraining() throws IOException, InterruptedException {
         VirtualEnvironmentRunner veRunner = getVirtualEnvironmentRunner();
 
         // This is the list of commands after the 'python' call
-        List<String> cellposeArguments = new ArrayList<>(Arrays.asList("-W", "ignore", "-m", runCommand));
+        List<String> cellposeArguments = new ArrayList<>(Arrays.asList( "-Xutf8", "-W", "ignore", "-m", runCommand));
 
         cellposeArguments.add("--train");
 
@@ -930,8 +933,8 @@ private void runTraining() throws IOException, InterruptedException {
             }
         });
 
-
-        cellposeArguments.add("--use_gpu");
+        // Some people may deactivate this...
+        if( this.useGPU ) cellposeArguments.add("--use_gpu");
 
         cellposeArguments.add("--verbose");
 
@@ -1364,6 +1367,7 @@ private Collection<CandidateObject> readObjectsFromFile(TileFile tileFile) throw
             }
         }
         // Ignore the IDs, because they will be the same across different images, and we don't really need them
+        if(candidates.isEmpty()) return Collections.emptyList();
         return candidates.values();
     }
 
diff --git a/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java b/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
index ece4ae1..0f4179a 100644
--- a/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
+++ b/src/main/java/qupath/ext/biop/cmd/VirtualEnvironmentRunner.java
@@ -28,7 +28,7 @@ public class VirtualEnvironmentRunner {
     private final EnvType envType;
     private WatchService watchService;
     private String name;
-    private String environmentNameOrPath;
+    private String pythonPath;
 
     private List<String> arguments;
 
@@ -61,7 +61,7 @@ public String toString() {
     }
 
     public VirtualEnvironmentRunner(String environmentNameOrPath, EnvType type, String name) {
-        this.environmentNameOrPath = environmentNameOrPath;
+        this.pythonPath = environmentNameOrPath;
         this.envType = type;
         this.name = name;
         if (envType.equals(EnvType.OTHER))
@@ -84,27 +84,29 @@ private List<String> getActivationCommand() {
             case CONDA:
                 switch (platform) {
                     case WINDOWS:
-                        cmd.addAll(Arrays.asList("CALL", "conda.bat", "activate", environmentNameOrPath, "&", "python"));
+                        // Adjust path to the folder with the env name based on the python location. On Windows it's at the root of the environment
+                        cmd.addAll(Arrays.asList("CALL", "conda.bat", "activate", new File(pythonPath).getParent(), "&", "python"));
                         break;
                     case UNIX:
                     case OSX:
-                        cmd.addAll(Arrays.asList("conda", "activate", environmentNameOrPath, ";", "python"));
+                        // Adjust path to the folder with the env name based on the python location. In Linux/MacOS it's in the 'bin' sub folder
+                        cmd.addAll(Arrays.asList("conda", "activate", new File(pythonPath).getParentFile().getParent(), ";", "python"));
                         break;
                 }
                 break;
             case VENV:
                 switch (platform) {
                     case WINDOWS:
-                        cmd.add(new File(environmentNameOrPath, "Scripts/python").getAbsolutePath());
+                        cmd.add(new File(pythonPath, "Scripts/python").getAbsolutePath());
                         break;
                     case UNIX:
                     case OSX:
-                        cmd.add(new File(environmentNameOrPath, "bin/python").getAbsolutePath());
+                        cmd.add(new File(pythonPath, "bin/python").getAbsolutePath());
                         break;
                 }
                 break;
             case EXE:
-                cmd.add(environmentNameOrPath);
+                cmd.add(pythonPath);
                 break;
             case OTHER:
                 return null;
@@ -123,8 +125,8 @@ public void setArguments(List<String> arguments) {
 
     /**
      * This builds, runs the command and outputs it to the logger as it is being run
-     * @param wait whether to wait for the process to end or not before exiting this command
-     * @throws IOException          // In case there is an issue starting the process
+     * @param waitUntilDone whether to wait for the process to end or not before exiting this method
+     * @throws IOException  in case there is an issue with the process
      */
     public void runCommand(boolean waitUntilDone) throws IOException {