Skip to content

Commit

Permalink
Merge pull request #4478 from broadinstitute/removeduplicate-mito-con…
Browse files Browse the repository at this point in the history
…straint

remove mito highConstraintRegion
  • Loading branch information
hanars authored Nov 12, 2024
2 parents e2293f3 + faa48b7 commit 60f720e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 12 deletions.
3 changes: 0 additions & 3 deletions hail_search/queries/mito.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,6 @@ class MitoHailTableQuery(BaseHailTableQuery):
CORE_FIELDS = BaseHailTableQuery.CORE_FIELDS + ['rsid']
MITO_ANNOTATION_FIELDS = {
'commonLowHeteroplasmy': lambda r: r.common_low_heteroplasmy,
'highConstraintRegion': (
lambda r: r.high_constraint_region if hasattr(r, 'high_constraint_region') else r.high_constraint_region_mito
),
'mitomapPathogenic': lambda r: r.mitomap.pathogenic,
}
BASE_ANNOTATION_FIELDS = {
Expand Down
3 changes: 0 additions & 3 deletions hail_search/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -915,7 +915,6 @@
'mlc': 3.38874,
},
'commonLowHeteroplasmy': False,
'highConstraintRegion': True,
'mitomapPathogenic': None,
'clinvar': {
'alleleId': 677748,
Expand Down Expand Up @@ -969,7 +968,6 @@
'mlc': None,
},
'commonLowHeteroplasmy': False,
'highConstraintRegion': True,
'mitomapPathogenic': None,
'clinvar': None,
'transcripts': {
Expand Down Expand Up @@ -1016,7 +1014,6 @@
'mlc': 0.7514,
},
'commonLowHeteroplasmy': True,
'highConstraintRegion': False,
'mitomapPathogenic': True,
'clinvar': {
'alleleId': 150280,
Expand Down
6 changes: 0 additions & 6 deletions ui/shared/components/panel/variants/Annotations.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -638,12 +638,6 @@ const Annotations = React.memo(({ variant, mainGeneId, showMainGene, transcripts
/>
</span>
)}
{variant.highConstraintRegion && (
<span>
<HorizontalSpacer width={12} />
<Label color="red" horizontal size="tiny">High Constraint Region</Label>
</span>
)}
{nonMajorConsequences.length > 0 && (
<div>
<b>Additonal VEP consequences: &nbsp;</b>
Expand Down

0 comments on commit 60f720e

Please sign in to comment.