Skip to content

atarva catalog - not skipping the internal repeats#341

Open
avvaruakshay wants to merge 1 commit into
mainfrom
catalog-script
Open

atarva catalog - not skipping the internal repeats#341
avvaruakshay wants to merge 1 commit into
mainfrom
catalog-script

Conversation

@avvaruakshay

Copy link
Copy Markdown
Contributor

Major Changes

  • Removed the continue statement for not adding internal repeats in atarva catalog
  • SCA37_DAB1 locus seems to have wrong number of count (repeat units) in the locus structure in the STRchive-loci.json. I was originally 7 but the locus has 15 in hg38.

@netlify

netlify Bot commented Apr 16, 2026

Copy link
Copy Markdown

Deploy Preview for strchive ready!

Name Link
🔨 Latest commit 0732d1f
🔍 Latest deploy log https://app.netlify.com/projects/strchive/deploys/69e1360283eacf00087178c5
😎 Deploy Preview https://deploy-preview-341--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.

@avvaruakshay avvaruakshay requested a review from hdashnow April 16, 2026 20:23
@hdashnow

Copy link
Copy Markdown
Member

Something hasn't quite worked here, and it's likely my fault for having a function that does too much stuff as it's used to generate other formats. I suggest adding atarva to the ref-alleles script so we can see exactly what is changing. It is likely necessary to break out some of the functionality so that atarva can have different rules from other methods.

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.

2 participants