Skip to content

Commit

Permalink
Merge pull request #323 from VariantEffect/bugfix/bencap/322/display-…
Browse files Browse the repository at this point in the history
…nucleotide-heatmap-for-regulatory-targets

Fixed: Nucleotide heatmap was not displayed for regulatory targets.
  • Loading branch information
bencap authored Nov 22, 2024
2 parents b29770f + b1b246f commit abed5de
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/components/ScoreSetHeatmap.vue
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,9 @@ export default {
return [...this.simpleVariants || [], ...this.wtVariants || []]
},
isNucleotideHeatmap: function() {
return _.get(this.scoreSet, 'targetGenes[0].category') === 'other_noncoding'
const targetCategory = _.get(this.scoreSet, 'targetGenes[0].category')
const proteinVariantsAreDefined = this.scores.every((elem) => elem.hgvs_pro !== null)
return !proteinVariantsAreDefined && (targetCategory === 'other_noncoding' || targetCategory == "regulatory")
},
heatmapRows: function() {
return this.isNucleotideHeatmap ? HEATMAP_NUCLEOTIDE_ROWS : HEATMAP_AMINO_ACID_ROWS
Expand Down

0 comments on commit abed5de

Please sign in to comment.