Skip to content

Commit

Permalink
Fix don't hide GTR option if CAT subst rates
Browse files Browse the repository at this point in the history
Assume -m GTRGAMMA if non-GTR model override

Fixes #257
  • Loading branch information
danieledler committed May 9, 2022
1 parent a310090 commit 0b2fbb6
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions src/app/store/Run.js
Original file line number Diff line number Diff line change
Expand Up @@ -392,10 +392,6 @@ class SubstitutionMatrix extends Option {
this.run.dataType === 'binary' ||
this.run.dataType === 'protein' ||
this.run.dataType === 'multistate' ||
((this.run.dataType === 'dna' ||
this.run.dataType === 'rna' ||
this.run.dataType === 'nucleotide') &&
this.run.substitutionRate.isCAT) ||
this.run.usesRaxmlNg
);
}
Expand Down Expand Up @@ -455,6 +451,12 @@ class SubstitutionRate extends Option {
this.run.usesRaxmlNg
);
}
@computed get cmdValue() {
if (this.run.substitutionMatrix.notGTR) {
return 'GAMMA';
}
return this.value;
}
@computed get isCAT() {
return this.value === 'CAT';
}
Expand Down Expand Up @@ -1056,7 +1058,7 @@ class Run extends StoreBase {
let model =
this.substitutionAscertainment.cmdValue +
this.substitutionMatrix.cmdValue +
this.substitutionRate.value +
this.substitutionRate.cmdValue +
this.substitutionI.cmdValue;
if (this.dataType === 'protein') {
model += this.alignments[0].aaMatrixName;
Expand Down

0 comments on commit 0b2fbb6

Please sign in to comment.