Skip to content

Frequency of ALL splits#250

Merged
ms609 merged 6 commits intomainfrom
invSplitFreq
Feb 16, 2026
Merged

Frequency of ALL splits#250
ms609 merged 6 commits intomainfrom
invSplitFreq

Conversation

@ms609
Copy link
Owner

@ms609 ms609 commented Feb 16, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Feb 16, 2026

⚠️ This benchmark result is outdated. See the latest comment below.

Performance benchmark results

Call Status Change Time (ms)
as.Splits(bigTrees) ⚪ NSD -2.65% 29 →
29.8, 30
as.Splits(someTrees) ⚪ NSD 0.37% 9.98 →
9.91, 9.97
Consensus(forest1k.888, check = FALSE) ⚪ NSD 0.81% 85.8 →
85.6, 84.6
Consensus(forest201.80, check = FALSE) ⚪ NSD 1.29% 3.95 →
3.89, 3.9
Consensus(forest21.260, 0.5, FALSE) ⚪ NSD 0.25% 1.12 →
1.12, 1.12
Consensus(forest21.260) ⚪ NSD 0.32% 1.12 →
1.12, 1.11
Consensus(forestMaj, 0.5, FALSE) ⚪ NSD 0.72% 2.84 →
2.82, 2.82
DropTip(tr2000, 5) ⚪ NSD 0.07% 25.5 →
25.4, 25.6
DropTip(tr80, 5) ⚪ NSD -0.43% 0.0883 →
0.0896, 0.0879
DropTip(unlen2k, 5) ⚪ NSD 1.69% 0.201 →
0.197, 0.198
DropTip(unlen80, 5) ⚪ NSD 0.93% 0.0342 →
0.0338, 0.0338
lapply(bigSplits, as.phylo) ⚪ NSD -0.52% 29.3 →
29.3, 29.5
lapply(someSplits, as.phylo) ⚪ NSD -1.74% 11.7 →
12, 11.9
PathLengths(tr2000, full = TRUE) ⚪ NSD 0.61% 24 →
23.8, 23.8
PathLengths(tr80, full = TRUE) ⚪ NSD 3.54% 0.0775 →
0.0764, 0.0337
PathLengths(tr80Unif, full = TRUE) ⚪ NSD -137.37% 0.0319 →
0.0781, 0.0327
RootTree(tr2000, 5) ⚪ NSD -0.16% 0.311 →
0.31, 0.312
RootTree(tr80, c("t3", "t36")) ⚪ NSD -0.31% 0.0558 →
0.0559, 0.0561
RootTree(tr80, "t3") ⚪ NSD 0.68% 0.0395 →
0.0393, 0.0392
RootTree(tr80, "t30") ⚪ NSD 0.18% 0.0394 →
0.0394, 0.0393
RootTree(unlen2k, 5) ⚪ NSD -0.92% 0.295 →
0.3, 0.296
RootTree(unlen80, c("t3", "t36")) ⚪ NSD 0.57% 0.052 →
0.0518, 0.0516
RootTree(unlen80, "t3") ⚪ NSD 0.88% 0.035 →
0.0349, 0.0345
RootTree(unlen80, "t30") ⚪ NSD 0.09% 0.0348 →
0.0348, 0.0347
TreeDist::RobinsonFoulds(forest201.80) ⚪ NSD 1.51% 14.8 →
14.6, 15.1
TreeDist::RobinsonFoulds(forest21.888) ⚪ NSD 0.37% 2.89 →
2.89, 2.86
TreeTools:::path_lengths(tr80$edge, tr80$edge.length, FALSE) ⚪ NSD -0.08% 0.0696 →
0.0699, 0.0695
TreeTools:::postorder_order(bal40) ⚪ NSD -0.82% 0.00132 →
0.00133, 0.00133
TreeTools:::postorder_order(bal40k) ⚪ NSD 0.64% 0.316 →
0.312, 0.316
TreeTools:::postorder_order(dbal40) ⚪ NSD -0.8% 0.0015 →
0.00152, 0.0015
TreeTools:::postorder_order(dbal40k) ⚪ NSD -0.34% 2.22 →
2.22, 2.23
TreeTools:::postorder_order(dpec40) ⚪ NSD -0.9% 0.00254 →
0.00257, 0.00255
TreeTools:::postorder_order(dpec40k) 🟣 ~same -1% 4200 →
4240, 4240
TreeTools:::postorder_order(drnd80) ⚪ NSD -0.19% 0.00408 →
0.0041, 0.00405
TreeTools:::postorder_order(nbal40) ⚪ NSD -1.45% 0.00186 →
0.00188, 0.00189
TreeTools:::postorder_order(nbal40k) ⚪ NSD -0.03% 2.42 →
2.41, 2.43
TreeTools:::postorder_order(npec40) ⚪ NSD -0.81% 0.00285 →
0.00288, 0.00286
TreeTools:::postorder_order(npec40k) 🟣 ~same -0.92% 4210 →
4270, 4240
TreeTools:::postorder_order(nrnd80) ⚪ NSD -0.91% 0.00475 →
0.00475, 0.00483
TreeTools:::postorder_order(pec40) ⚪ NSD -1.44% 0.00133 →
0.00137, 0.00134
TreeTools:::postorder_order(pec40k) ⚪ NSD 0% 0.312 →
0.31, 0.314
TreeTools:::postorder_order(rnd80) ⚪ NSD 0.06% 0.00161 →
0.0016, 0.00161

@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.20%. Comparing base (49064ee) to head (8e857c6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
+ Coverage   96.19%   96.20%   +0.01%     
==========================================
  Files          75       75              
  Lines        5466     5481      +15     
==========================================
+ Hits         5258     5273      +15     
  Misses        208      208              

☔ 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.

@github-actions
Copy link

Performance benchmark results

Call Status Change Time (ms)
as.Splits(bigTrees) ⚪ NSD -4.97% 21.8 →
23.2, 21.1
as.Splits(someTrees) ⚪ NSD 0.03% 11.3 →
11, 11.3
Consensus(forest1k.888, check = FALSE) ⚪ NSD -1.45% 95.5 →
97.4, 96.9
Consensus(forest201.80, check = FALSE) ⚪ NSD -0.56% 4 →
4.04, 4.01
Consensus(forest21.260, 0.5, FALSE) ⚪ NSD -0.67% 1.22 →
1.23, 1.23
Consensus(forest21.260) ⚪ NSD -1.04% 1.23 →
1.25, 1.24
Consensus(forestMaj, 0.5, FALSE) ⚪ NSD -0.53% 2.94 →
2.97, 2.95
DropTip(tr2000, 5) 🟣 ~same 2.95% 17 →
16.5, 16.5
DropTip(tr80, 5) ⚪ NSD 1.98% 0.106 →
0.103, 0.105
DropTip(unlen2k, 5) ⚪ NSD 1.26% 0.215 →
0.21, 0.214
DropTip(unlen80, 5) ⚪ NSD 1.45% 0.0406 →
0.0401, 0.04
lapply(bigSplits, as.phylo) ⚪ NSD 0.13% 31.2 →
30.9, 31.4
lapply(someSplits, as.phylo) ⚪ NSD 0.67% 14 →
13.9, 14
PathLengths(tr2000, full = TRUE) ⚪ NSD 1.78% 16.5 →
16.2, 16.2
PathLengths(tr80, full = TRUE) ⚪ NSD -5.36% 0.0286 →
0.0431, 0.0278
PathLengths(tr80Unif, full = TRUE) ⚪ NSD -7.89% 0.0297 →
0.0703, 0.0295
RootTree(tr2000, 5) ⚪ NSD 0.19% 0.391 →
0.39, 0.39
RootTree(tr80, c("t3", "t36")) ⚪ NSD 0.17% 0.0699 →
0.0692, 0.0702
RootTree(tr80, "t3") ⚪ NSD 1.16% 0.0499 →
0.0496, 0.0489
RootTree(tr80, "t30") ⚪ NSD 0.52% 0.0501 →
0.0501, 0.0494
RootTree(unlen2k, 5) ⚪ NSD 0.44% 0.324 →
0.322, 0.324
RootTree(unlen80, c("t3", "t36")) ⚪ NSD 0.19% 0.0638 →
0.0632, 0.0641
RootTree(unlen80, "t3") ⚪ NSD 0.49% 0.0427 →
0.0427, 0.0422
RootTree(unlen80, "t30") ⚪ NSD 0.47% 0.043 →
0.0429, 0.0426
TreeDist::RobinsonFoulds(forest201.80) ⚪ NSD 1.37% 16.7 →
16.6, 16.3
TreeDist::RobinsonFoulds(forest21.888) ⚪ NSD -0.07% 3.37 →
3.38, 3.35
TreeTools:::path_lengths(tr80$edge, tr80$edge.length, FALSE) ⚪ NSD -1.62% 0.102 →
0.105, 0.102
TreeTools:::postorder_order(bal40) ⚪ NSD -8.02% 0.00174 →
0.00174, 0.00194
TreeTools:::postorder_order(bal40k) ⚪ NSD 0.68% 0.43 →
0.427, 0.428
TreeTools:::postorder_order(dbal40) ⚪ NSD -6.85% 0.00188 →
0.00185, 0.00207
TreeTools:::postorder_order(dbal40k) ⚪ NSD -0.94% 2.34 →
2.35, 2.38
TreeTools:::postorder_order(dpec40) ⚪ NSD -4.55% 0.0031 →
0.00306, 0.00329
TreeTools:::postorder_order(dpec40k) ⚪ NSD 0.01% 3610 →
3610, 3610
TreeTools:::postorder_order(drnd80) ⚪ NSD -0.02% 0.00516 →
0.00514, 0.00519
TreeTools:::postorder_order(nbal40) ⚪ NSD -3.62% 0.00223 →
0.00218, 0.00239
TreeTools:::postorder_order(nbal40k) ⚪ NSD 0.36% 2.41 →
2.4, 2.4
TreeTools:::postorder_order(npec40) ⚪ NSD -3.54% 0.00339 →
0.00336, 0.00355
TreeTools:::postorder_order(npec40k) ⚪ NSD 0.25% 3640 →
3630, 3620
TreeTools:::postorder_order(nrnd80) ⚪ NSD -0.16% 0.00571 →
0.00572, 0.00572
TreeTools:::postorder_order(pec40) ⚪ NSD -9.8% 0.00172 →
0.00171, 0.00194
TreeTools:::postorder_order(pec40k) ⚪ NSD 0.19% 0.429 →
0.425, 0.43
TreeTools:::postorder_order(rnd80) ⚪ NSD -0.46% 0.00217 →
0.00217, 0.00219

@ms609 ms609 merged commit 2d350fe into main Feb 16, 2026
34 checks passed
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.

1 participant