Skip to content

Trimming tree#140

Merged
smasongarrison merged 8 commits intodev_mainfrom
trimming_tree
Mar 27, 2026
Merged

Trimming tree#140
smasongarrison merged 8 commits intodev_mainfrom
trimming_tree

Conversation

@smasongarrison
Copy link
Copy Markdown
Member

This pull request introduces new functionality for identifying and trimming "leaf" nodes from pedigree datasets. The main additions are the findLeaves and trimPedigree functions, which allow users to systematically detect and remove individuals who do not substantially affect the structure of the pedigree tree. Comprehensive documentation for both functions has also been added.

New pedigree leaf detection and trimming functionality:

  • Added findLeaves function to identify "leaf" nodes in a pedigree, with flexible options for which types of leaves to include and the ability to protect individuals based on phenotype data. (R/trimPedigree.R, NAMESPACE, man/findLeaves.Rd) [1] [2] [3]
  • Added trimPedigree function to iteratively remove leaf nodes from a pedigree, supporting user-specified removals, phenotype-based protection, and stopping conditions such as minimum pedigree size and maximum iterations. (R/trimPedigree.R, NAMESPACE, man/trimPedigree.Rd) [1] [2] [3]

Documentation:

  • Generated roxygen2 documentation for both findLeaves and trimPedigree, including usage, argument details, return values, and examples. (man/findLeaves.Rd, man/trimPedigree.Rd) [1] [2]

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 97.22222% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.65%. Comparing base (8a7ce30) to head (9844020).
⚠️ Report is 1 commits behind head on dev_main.

Files with missing lines Patch % Lines
R/trimPedigree.R 97.22% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           dev_main     #140      +/-   ##
============================================
+ Coverage     84.47%   84.65%   +0.18%     
============================================
  Files            30       31       +1     
  Lines          4973     5045      +72     
============================================
+ Hits           4201     4271      +70     
- Misses          772      774       +2     

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

@smasongarrison smasongarrison merged commit c677637 into dev_main Mar 27, 2026
6 of 7 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