Skip to content

Lowlevel pair coalescence quantiles#2976

Merged
mergify[bot] merged 1 commit intotskit-dev:mainfrom
nspope:pair-coalescence-quantiles
Aug 28, 2024
Merged

Lowlevel pair coalescence quantiles#2976
mergify[bot] merged 1 commit intotskit-dev:mainfrom
nspope:pair-coalescence-quantiles

Conversation

@nspope
Copy link
Contributor

@nspope nspope commented Aug 9, 2024

C implementation of pair_coalescence_quantiles

@codecov
Copy link

codecov bot commented Aug 9, 2024

Codecov Report

Attention: Patch coverage is 98.87640% with 2 lines in your changes missing coverage. Please review.

Project coverage is 89.72%. Comparing base (0403de0) to head (b551a95).
Report is 1 commits behind head on main.

Files Patch % Lines
python/_tskitmodule.c 97.64% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2976      +/-   ##
==========================================
+ Coverage   89.66%   89.72%   +0.05%     
==========================================
  Files          29       29              
  Lines       30435    30609     +174     
  Branches     5905     5938      +33     
==========================================
+ Hits        27290    27464     +174     
  Misses       1799     1799              
  Partials     1346     1346              
Flag Coverage Δ
c-tests 86.43% <100.00%> (+0.07%) ⬆️
lwt-tests 80.78% <ø> (ø)
python-c-tests 88.92% <97.64%> (+0.10%) ⬆️
python-tests 99.01% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
c/tskit/core.c 95.67% <100.00%> (+0.03%) ⬆️
c/tskit/core.h 100.00% <ø> (ø)
c/tskit/trees.c 90.83% <100.00%> (+0.15%) ⬆️
python/tskit/trees.py 98.76% <100.00%> (+0.01%) ⬆️
python/_tskitmodule.c 88.92% <97.64%> (+0.10%) ⬆️

@nspope nspope marked this pull request as ready for review August 9, 2024 04:00
@nspope
Copy link
Contributor Author

nspope commented Aug 9, 2024

This is ready for a look @jeromekelleher

@nspope nspope force-pushed the pair-coalescence-quantiles branch from cc507c0 to 8c4d3d3 Compare August 9, 2024 05:49
Copy link
Member

@jeromekelleher jeromekelleher left a comment

Choose a reason for hiding this comment

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

LGTM! Happy to merge when you are.

@nspope nspope force-pushed the pair-coalescence-quantiles branch from 8c4d3d3 to b551a95 Compare August 28, 2024 02:00
@nspope
Copy link
Contributor Author

nspope commented Aug 28, 2024

Great, let's get it in!

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