Skip to content

[WIP] Document all imported kinship functions#89

Merged
smasongarrison merged 4 commits intokinship2portfrom
copilot/document-kinship-functions
Nov 7, 2025
Merged

[WIP] Document all imported kinship functions#89
smasongarrison merged 4 commits intokinship2portfrom
copilot/document-kinship-functions

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 7, 2025

  • Add roxygen2 documentation for kinship2_alignped1 (pedigree alignment helper - processes single subject and spouses)
  • Add roxygen2 documentation for kinship2_alignped2 (pedigree alignment helper - processes sibling groups)
  • Add roxygen2 documentation for kinship2_alignped3 (pedigree alignment helper - merges two aligned pedigree structures)
  • Add roxygen2 documentation for kinship2_alignped4 (pedigree alignment helper - computes optimal horizontal spacing)
  • Add roxygen2 documentation for kinship2_bitSize (calculates bit size of a pedigree)
  • Add roxygen2 documentation for kinship2_check.hint (validates kinship2 hints for consistency)
  • Add roxygen2 documentation for kinship2_autohint (automatically generates alignment hints)
  • Add roxygen2 documentation for kinship2_align.pedigree (main pedigree alignment function)
  • Add roxygen2 documentation for kinship2_kindepth (calculates depth/generation of subjects)
  • Fix typo in kinship2_bitSize error message (pegigree -> pedigree)
  • All 9 kinship2 functions now have comprehensive roxygen2 documentation
  • Ready for final validation

Summary

All kinship2 imported functions have been documented with roxygen2 comments:

  • 9 functions documented
  • 88 roxygen2 tags added
  • All parameters and return values documented
  • Functions marked as internal helpers
  • Typo fix applied
  • Documentation will be built by GitHub Actions R CMD check
Original prompt

This section details on the original issue you should resolve

<issue_title>Need to document all the imported kinship functions</issue_title>
<issue_description>Need to document all the imported kinship functions. These are all named kinship2_FOO</issue_description>

<agent_instructions>Please help me document all of the kinship 2 files I'm working on. They need to work with roxygen</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
@smasongarrison smasongarrison marked this pull request as ready for review November 7, 2025 18:58
smasongarrison and others added 2 commits November 7, 2025 14:00
Co-authored-by: smasongarrison <6001608+smasongarrison@users.noreply.github.com>
@smasongarrison smasongarrison merged commit bf4eb73 into kinship2port Nov 7, 2025
Copilot AI requested a review from smasongarrison November 7, 2025 19:01
@smasongarrison smasongarrison deleted the copilot/document-kinship-functions branch November 7, 2025 22:40
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