Disable modeltest on Win 7

Author: jtklein

src/app/AlignmentCard.js

@@ -158,6 +158,7 @@ function _ModelTestButton({ alignment }) {
       }}
       loading={alignment.modeltestLoading}
       onClick={alignment.runModelTest}
+      disabled={alignment.modeltestDisabled}
     >
       Run ModelTest
     </LoadingButton>

src/app/store/Alignment.js

@@ -13,6 +13,8 @@ import * as raxmlNgSettings from '../../settings/raxmlng';
 import StoreBase from './StoreBase';
 import Partition, { FinalPartition } from './Partition';
 import { getFinalDataType } from '../../common/typecheckAlignment';
+import os from 'os';
+import electronutil from 'electron-util';
 
 const raxmlMatrixOptions = raxmlSettings.matrixOptions;
 const raxmlNgModelOptions = raxmlNgSettings.modelOptions;
@@ -237,6 +239,10 @@ class Alignment extends StoreBase {
     this.ngAscertainmentBias = new RaxmlNgModelASC(this);
     this.partition = new Partition(this);
 
+    const [majorVersion, minorVersion] = os.release().split('.');
+    this.modeltestDisabled =
+      electronutil.is.windows && majorVersion === '6' && minorVersion === '1';
+
     this.listen();
   }
 
@@ -387,7 +393,7 @@ class Alignment extends StoreBase {
     }
     // I couldnt find information if ModelTest checks for ASC_
     this.run.substitutionAscertainment.setValue('none');
-    
+
     // Check for invariant sites
     if (/I$/.test(model)) {
       this.run.substitutionI.setValue('+I (ML estimate)');

src/app/store/Run.js

@@ -34,7 +34,8 @@ const winBinaries =
   majorVersion === '6' && minorVersion === '1'
     ? [
         // TODO: add raxml ng windows exe
-        { name: 'modeltest-ng.exe', version: '0.1.7' },
+        // TODO: Crashes on Win7, disabled for now
+        // { name: 'modeltest-ng.exe', version: '0.1.7' },
         { name: 'raxmlHPC_Win7.exe', multithreaded: false, version: '8.2.10' },
         {
           name: 'raxmlHPC-SSE3_Win7.exe',
@@ -711,7 +712,7 @@ class Run extends StoreBase {
     this.outputNamePlaceholder = `${id}`;
     this.atomAfterRun = createAtom('AfterRun');
     this.atomFinished = createAtom('finished');
-    this.modeltestName = binaries.filter((b) => b.name.includes('modeltest'))[0].name;
+    this.modeltestName = binaries.filter((b) => b.name.includes('modeltest'))[0]?.name;
   }
 
   id = 0;
@@ -990,7 +991,6 @@ class Run extends StoreBase {
   }
 
   @computed get modelTestIsRunningOnAlignment() {
-    // return this.alignments.some(alignment => alignment.modeltestLoading);
     const running = this.alignments.some(
       (alignment) => alignment.modeltestLoading
     );