Skip to content

Conversation

@jonbrenas
Copy link
Collaborator

@jonbrenas jonbrenas commented Mar 13, 2025

Partially addresses #652 .

Created 2 functions inversions_frequencies and inversion_frequencies_advanced that should be compatible with other frequencies functions.

I still have to add tests (and they will be integration tests for now as we have not developed the simulated data for the karyotypes yet).

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov
Copy link

codecov bot commented Apr 13, 2025

Codecov Report

Attention: Patch coverage is 14.56954% with 129 lines in your changes missing coverage. Please review.

Project coverage is 93.56%. Comparing base (0197957) to head (1f09857).
Report is 19 commits behind head on master.

Files with missing lines Patch % Lines
malariagen_data/anoph/inversion_frq.py 13.42% 129 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #732      +/-   ##
==========================================
- Coverage   96.13%   93.56%   -2.57%     
==========================================
  Files          47       48       +1     
  Lines        4683     4834     +151     
==========================================
+ Hits         4502     4523      +21     
- Misses        181      311     +130     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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