Skip to contents

Contributing to mfrmr

Source: CONTRIBUTING.md

Thanks for helping improve mfrmr.

Before you start

  • Search existing issues and pull requests first.
  • If behavior changes are large, open an issue before implementation.
  • Keep changes focused and reviewable.

Development setup

# from package root
install.packages(c("devtools", "roxygen2", "testthat"))
devtools::document()
devtools::test()
devtools::check(args = c("--no-manual"), document = FALSE)

Coding guidelines

  • Prefer readable, explicit code over compact but opaque code.
  • Keep public API names descriptive.
  • For user-facing behavior changes, update docs and examples in the same PR.
  • Use base R plotting defaults in this package unless there is a strong reason not to.

Testing expectations

  • Add or update tests for every user-visible change.
  • Keep tests deterministic (fixed seeds, no network calls).
  • Avoid writing plot files during tests (use draw = FALSE unless plotting is under test).

Documentation expectations

  • Update roxygen comments when function arguments/returns change.
  • Run devtools::document() before committing.
  • If workflow changes, update README and/or vignette accordingly.

Examples and timing policy

CRAN examples are smoke checks, not the full validation suite. Keep Rd examples short enough to run on slower Windows check hosts, and move realistic multi-step analyses to README/vignettes or non-CRAN tests.

  • Use example_core for ordinary fitting, plotting, and reporting examples. Use example_bias only when a non-null DFF/bias signal is needed.
  • Prefer method = "JML", maxit = 30, and diagnose_mfrm(..., residual_pca = "none") in standard Rd examples.
  • Wrap multi-fit workflows, MML examples, recovery simulations, design simulations, external-Suggests examples, and long reporting pipelines in \donttest{} unless the function cannot be demonstrated otherwise.
  • When an MML example must run in standard examples, set a small quad_points value and explain that it is an exploratory speed setting.
  • Use draw = FALSE in examples that only need to demonstrate returned plot payloads.
  • Do not shrink example data below a meaningful many-facet structure just to satisfy CRAN timing. Reduce what CRAN executes; keep realistic examples in vignettes and in the full NOT_CRAN=true test run.
  • CRAN-time testthat is intentionally limited by tests/testthat.R; run the complete suite locally/CI with NOT_CRAN=true.

Pull request checklist

Reporting bugs

Please include:

  • minimal reproducible example,
  • session info (sessionInfo()),
  • expected vs observed behavior,
  • relevant data schema (without sensitive content).