Skip to content

Compute and plot inversion frequencies#1282

Open
31puneet wants to merge 21 commits intomalariagen:masterfrom
31puneet:652-karyotype-frequencies
Open

Compute and plot inversion frequencies#1282
31puneet wants to merge 21 commits intomalariagen:masterfrom
31puneet:652-karyotype-frequencies

Conversation

@31puneet
Copy link
Copy Markdown
Contributor

This PR introduces the inversion_frequencies to resolve #652, and replace PR #732. I have added the following refactoring and tests to finalize the feature:

  • Implementes inversion_frequencies and inversion_frequencies_advanced analogous to SNP calculations.
  • Added unit tests extending test_inversion_frq.py

@31puneet
Copy link
Copy Markdown
Contributor Author

Hi @jonbrenas
Do you suggest any changes over here ?

Copy link
Copy Markdown
Collaborator

@jonbrenas jonbrenas left a comment

Choose a reason for hiding this comment

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

If tests are added in test_inversion_frq.py, we don't need the whole test suite in test_ag3.py.

@31puneet
Copy link
Copy Markdown
Contributor Author

If tests are added in test_inversion_frq.py, we don't need the whole test suite in test_ag3.py.

Done!

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.

Karyotype frequencies

2 participants