Skip to content

Fixes bug at chromosome boundaries, refactors the code, changed het_ratio calculation#6

Open
biomonika wants to merge 1 commit intovollgerlab:masterfrom
biomonika:chromosome_boundaries
Open

Fixes bug at chromosome boundaries, refactors the code, changed het_ratio calculation#6
biomonika wants to merge 1 commit intovollgerlab:masterfrom
biomonika:chromosome_boundaries

Conversation

@biomonika
Copy link

  • fixes the bug discovered by Emily Xu; when at the chromosome boundaries, sometimes the clustering of variants would go across, creating large regions that include the coordinates from the two original chromosomes
  • het_ratio is now the average across all second and first alleles (sum_second/(sum_first+sum_second))

…now the average across all second and first alleles
@mrvollger
Copy link
Collaborator

Thanks for the PR! But @glogsdon1 contributed this code to NucFreq so I'd like/need her input on these changes before accepting a PR.

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