Skip to content

RXR-2949: Clean up package #1

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.

Sign up for GitHub

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

Jump to bottom
Draft
mccarthy-m-g wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

RXR-2949: Clean up package #1

mccarthy-m-g wants to merge 10 commits into from

Conversation

@mccarthy-m-g
Copy link

@mccarthy-m-g mccarthy-m-g commented Dec 23, 2025 •
edited
Loading

This PR does some package clean up, specifically:

  • updated README + using README Rmd
  • using codecov in CI
  • using pkgdown
  • adding a package doc
  • cleaning up function code and improving documentation
  • adding tests
  • adding a NEWS file
  • adding MIT license

There are currently some outstanding TODOs, so I've marked this as a draft:

  • implementing codecov and pkgdown on GitHub (I either need repo permissions or you'll need to set up the remaining bits)
  • write tests for reformat_data_sdtm_to_modeling(). This function is currently a big monolith. It would be nice to refactor into smaller functions for readability and testing
  • write code for reformat_data_modeling_to_nca() + tests. This function currently exists but has no code within the function
  • several misc TODOs (search TODO in the project)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@roninsightrx roninsightrx Awaiting requested review from roninsightrx

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mccarthy-m-g