Code review comments.

broadinstitute · Dec 19, 2022 · b33ff46 · b33ff46
1 parent bd08e80
commit b33ff46
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/main/java/org/broadinstitute/hellbender/tools/walkers/vqsr/CNNVariantTrain.java b/src/main/java/org/broadinstitute/hellbender/tools/walkers/vqsr/CNNVariantTrain.java
@@ -135,12 +135,12 @@ public class CNNVariantTrain extends CommandLineProgram {
     @Override
     protected void onStartup() {
         PythonScriptExecutor.checkPythonEnvironmentForPackage("vqsr_cnn");
+        // Start the Python executor. This does not actually start the Python process, but fails if python can't be located
+        pythonExecutor = new PythonScriptExecutor(true);
     }
 
     @Override
     protected Object doWork() {
-        // Start the Python executor. This does not actually start the Python process, but fails if python can't be located
-        pythonExecutor = new PythonScriptExecutor(true);
 
         final Resource pythonScriptResource = new Resource("training.py", CNNVariantTrain.class);
         List<String> arguments = new ArrayList<>(Arrays.asList(

diff --git a/src/main/java/org/broadinstitute/hellbender/tools/walkers/vqsr/CNNVariantWriteTensors.java b/src/main/java/org/broadinstitute/hellbender/tools/walkers/vqsr/CNNVariantWriteTensors.java
@@ -122,12 +122,13 @@ public class CNNVariantWriteTensors extends CommandLineProgram {
     @Override
     protected void onStartup() {
         PythonScriptExecutor.checkPythonEnvironmentForPackage("vqsr_cnn");
+
+        // Start the Python executor. This does not actually start the Python process, but fails if python can't be located
+        pythonExecutor = new PythonScriptExecutor(true);
     }
 
     @Override
     protected Object doWork() {
-        // Start the Python executor. This does not actually start the Python process, but fails if python can't be located
-        pythonExecutor = new PythonScriptExecutor(true);
 
         final Resource pythonScriptResource = new Resource("training.py", CNNVariantWriteTensors.class);
         List<String> arguments = new ArrayList<>(Arrays.asList(