Skip to content

Add ep400 - #508

Merged
hdashnow merged 7 commits into
mainfrom
Add_EP400
Aug 14, 2026
Merged

Add ep400#508
hdashnow merged 7 commits into
mainfrom
Add_EP400

Conversation

@gaberbz

@gaberbz gaberbz commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

Summarize the changes

Fixes: #480

Major Changes

  • Added SCA_EP400

Minor Changes

Checklist

  • All changes are well summarized
  • Check all tests pass
  • Check that the website preview looks good
  • Update the STRchive version in CITATION.cff, format X.Y.Z. If any major changes, increment Y. If only minor changes, increment Z. If the breaking change (rare), increment X.
  • Ask someone to review this PR

@netlify

netlify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Deploy Preview for strchive ready!

Name Link
🔨 Latest commit e26f78d
🔍 Latest deploy log https://app.netlify.com/projects/strchive/deploys/6a7f9ab866af8e0008294705
😎 Deploy Preview https://deploy-preview-508--strchive.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@gaberbz gaberbz closed this Jul 30, 2026
@gaberbz gaberbz reopened this Jul 30, 2026
@gaberbz
gaberbz marked this pull request as ready for review August 14, 2026 19:55
@gaberbz

gaberbz commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

This is ready for review. The main thing I am unsure of is how I described the differing locus definitions.

Update evidence and genomic coordinates for SCA_EP400

ep400 provisional

liftover

adding family 2

Update data

add family 2 info

Update data

fix

nomad

Update data

add description

Update data

ep400 nuance

Update data

add exon

fix

actually fix this time

clarify broad range

why is year a string and not an int?

Update data

show alternate locus definition

@hdashnow hdashnow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggested some rewording for details and have a few minor items that might need your follow-up

Comment thread data/STRchive-loci.json Outdated
"details": "Disease link reported in Danzi et al. Family 1 (father and daughter) had a longest pure tract of 56-58 repeats, while Family 2 (mother-son, mother ungenotyped) had 75 pure CAG repeats. STRchive is using the broad locus definition of chr12:132062524-132062611, where the reference locus structure is (CAG)6-(CAA)2-(CAG)14-(CAA)1-(CAG)4-(CAA)1-(CAG)1. A narrower locus definition of chr12:132062548-132062611 has also been used to describe this locus and results in a differently described pathogenic range [@doi:10.1101/2025.01.06.631535].",
"detection": "Long-read sequencing with targeted sanger confirmation has detected expansions in this locus [@doi:10.1101/2025.01.06.631535].",
"mechanism": "Unknown",
"mechanism_detail": "Polyglutamine expansion",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Citations for mechanism detail? Also, is there a value for "mechanism"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is not a value for mechanism. We only know the mechanisms of similar loci and the mechanism of other EP400 mutations. There isn't mechanistic information for this one. I'll delete the detail part since it might be misinterpreted as toxic polyQ GoF

Comment thread data/STRchive-loci.json
"typ_age_onset_min": null,
"typ_age_onset_max": null,
"details": "Disease link reported in Danzi et al. Family 1 (father and daughter) had a longest pure tract of 56-58 repeats, while Family 2 (mother-son, mother ungenotyped) had 75 pure CAG repeats. STRchive is using the broad locus definition of chr12:132062524-132062611, where the reference locus structure is (CAG)6-(CAA)2-(CAG)14-(CAA)1-(CAG)4-(CAA)1-(CAG)1. A narrower locus definition of chr12:132062548-132062611 has also been used to describe this locus and results in a differently described pathogenic range [@doi:10.1101/2025.01.06.631535].",
"detection": "Long-read sequencing with targeted sanger confirmation has detected expansions in this locus [@doi:10.1101/2025.01.06.631535].",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't they use ExpansionHunter on short reads as well? For controls? This was the stated reason for using the narrow locus definition I think.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They did in the controls. We do not know if short read is able to actually detect this expansion (from this paper at least) since the affected individuals were flagged from long read. I could put info that it is useful for controls, though the convention for other detection fields has been about detecting the expansion. What do you think?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's not in the other loci, I think it's probably fine to skip it.

@hdashnow

hdashnow commented Aug 14, 2026

Copy link
Copy Markdown
Member

Looks like the benign range should be 8-21 pure CAGs based on the population data (not sure if I'm reading those off the plot right!). If we assume the reference locus structure, that would be (CAG)6(CAA)2(CAG)8-21(CAA)1(CAG)4(CAA)(CAG)
23 - 36 motifs total.
Screenshot 2026-08-14 at 3 56 41 PM

Correction:
population 4-24 LPS in TRExploerer which is almost certainly the same data.
So adding 15 for the reference flanks that would be 19-39
https://trexplorer.broadinstitute.org/locus.html?#locusId=12-132062548-132062611-CAG&igvLoc=chr12:132062548-132062611&sc=isPathogenic&sd=DESC&showRs=1&q=chr12:132062525-132062611

Update:
I've edited the json to match.

@hdashnow hdashnow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All my comments have been addressed. Looks good to me.

@hdashnow
hdashnow merged commit 4d19531 into main Aug 14, 2026
6 checks passed
@hdashnow
hdashnow deleted the Add_EP400 branch August 14, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add EP400

2 participants