Closed mattsecrest closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 79.54%. Comparing base (
4e6b0b1
) to head (b7b7231
).:exclamation: Current head b7b7231 differs from pull request most recent head 40fb2ce. Consider uploading reports for the commit 40fb2ce to get more accurate results
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Please add \value to .Rd files regarding exported methods and explain the functions results in the documentation. Please write about the structure of the output (class) and also what the output means. (If a function does not return a value, please document that too, e.g. \value{No return value, called for side effects} or similar) Missing Rd-tags:
You have examples for unexported functions. Please either omit these examples or export these functions. Examples for unexported function
[x] c() in:
add_covariates.Rd baseline_covariates.Rd check_data_matrix_has_columns.Rd covariance_matrix.Rd create_analysis_obj.Rd create_baseline_object.Rd create_data_matrix.Rd create_data_simulation.Rd create_simulation_obj.Rd enrollment_constant.Rd generate-BaselineObject-method.Rd generate-DataSimObject-method.Rd get_covariate_constraints.Rd mcmc_sample.Rd plot.Rd rename_draws_covariates.Rd set_enrollment.Rd sim_covariate_list.Rd sim_data_list.Rd
"Using foo:::f instead of foo::f allows access to unexported objects. This is generally not recommended, as the semantics of unexported objects may be changed by the package author in routine maintenance."
[x] Used ::: in documentation:
man/check_data_matrix_has_columns.Rd man/get_covariate_constraints.Rd man/get_prior_string.Rd man/make_model_string_data.Rd etc.... Please omit one colon.
\dontrun{} should only be used if the example really cannot be executed (e.g. because of missing additional software, missing API keys, ...) by the user. That's why wrapping examples in \dontrun{} adds the comment ("# Not run:") as a warning for the user. Does not seem necessary. Please replace \dontrun with \donttest. Please unwrap the examples if they are executable in < 5 sec, or replace dontrun{} with \donttest{}.
We still see: Examples for unexported function
Please either omit these examples or export these functions.
Some code lines in examples are commented out. Please never do that. Ideally find toy examples that can be regularly executed and checked. Lengthy examples (> 5 sec), can be wrapped in \donttest{}. Examples in comments in:
Per Daniel Sjoberg's suggestions just wrapping in @noRd and pushing back on prior_gamma
, prior_normal
, and treatment_details
, which do not have commented out examples.
@danielinteractive @gravesti the package as-is was accepted to CRAN. Would someone be able to approve so I can move aditional tweaks to a new branch?
No response form Uwe yet let's see if this fixes things. cran-submit-2.pdf