-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add pair_coalescence_counts
method
#2920
Conversation
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2920 +/- ##
==========================================
+ Coverage 81.31% 81.60% +0.29%
==========================================
Files 48 48
Lines 37856 37426 -430
Branches 7295 7232 -63
==========================================
- Hits 30782 30542 -240
+ Misses 5653 5466 -187
+ Partials 1421 1418 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@benjeffery any idea what's going on with codecov here? |
Would it help to merge a small PR which doesn't do much to try and reset codecov's view of HEAD? |
The Codecov uploads are failing with:
Looks like this has been cropping up elsewhere? In particular, with PRs coming from forked repos. |
FWIW it sounds like bumping the codecov-action version from 3 to 4 may fix the issue: tskit/.github/workflows/tests.yml Line 159 in 9e1bf0d
there's a blurb in the v4 notes here,
|
Will try that - thanks for the pointer. |
…tion Add test against worked example Remove unused imports More efficient windowing Raise errors with invalid inputs Misc. fixes and tests; add time discretisation argument Test nonsuccinct case Fix negative times for time windows Delete accidental line copy Remove unneeded if
b431c6a
to
77bbe72
Compare
Closing in favor of #2915 |
Reprise of #2915