Skip to content

Commit

Permalink
Merge pull request #101 from rapidsurveys/dev
Browse files Browse the repository at this point in the history
address CRAN comments
  • Loading branch information
ernestguevarra authored Jan 21, 2025
2 parents e168147 + a25fc8f commit 95f0503
Show file tree
Hide file tree
Showing 12 changed files with 74 additions and 52 deletions.
1 change: 1 addition & 0 deletions .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,4 @@
^pkgdown$
^CODE_OF_CONDUCT\.md$
^README\.html$
^CRAN-SUBMISSION$
13 changes: 5 additions & 8 deletions DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: oldr
Title: An Implementation of Rapid Assessment Method for Older People
Version: 0.2.1
Version: 0.2.2
Authors@R: c(
person("Mark", "Myatt", comment = c(ORCID = "0000-0003-1119-1474"),
email = "[email protected]", role = c("aut", "cph")),
Expand All @@ -11,13 +11,10 @@ Authors@R: c(
person("HelpAge International", role = "cph"),
person("Elrha", role = "fnd"))
Description: An implementation of the Rapid Assessment Method for Older People
or RAM-OP <https://www.helpage.org/resource/rapid-assessment-method-for-older-people-ramop-manual/>.
It provides various functions that allow the user to design and plan the
assessment and analyse the collected data. RAM-OP provides accurate and
reliable estimates of the needs of older people. The method uses simple
procedures, in a short time frame (i.e. about two weeks including training,
data collection, data entry, and data analysis), and at considerably lower
cost than other methods.
or RAM-OP <https://www.helpage.org/resource/rapid-assessment-method-for-older-people-ramop-manual/>.
It provides various functions that allow the user to design and plan the
assessment and analyse the collected data. RAM-OP provides accurate and
reliable estimates of the needs of older people.
License: GPL-3
Depends: R (>= 4.1.0)
Imports:
Expand Down
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# oldr 0.2.1
# oldr 0.2.2

This is the first CRAN release of `{oldr}`.

Expand Down
14 changes: 7 additions & 7 deletions R/00-oldr.R
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@
#'
#' [HelpAge International](https://www.helpage.org/), VALID International, and
#' [Brixton Health](http://www.brixtonhealth.com), with financial
#' assistance from the
#' [Humanitarian Innovation Fund (HIF)](http://www.elrha.org/hif/home/), have
#' developed a **Rapid Assessment Method for Older People (RAM-OP)** that
#' provides accurate and reliable estimates of the needs of older people. The
#' method uses simple procedures, in a short time frame (i.e. about two weeks
#' including training, data collection, data entry, and data analysis), and at
#' assistance from the
#' [Humanitarian Innovation Fund (HIF)](http://www.elrha.org/hif/home/), have
#' developed a **Rapid Assessment Method for Older People (RAM-OP)** that
#' provides accurate and reliable estimates of the needs of older people. The
#' method uses simple procedures, in a short time frame (i.e. about two weeks
#' including training, data collection, data entry, and data analysis), and at
#' considerably lower cost than other methods.
#'
#' The **RAM-OP** method is based on the following principles:
Expand All @@ -21,7 +21,7 @@
#' prevalence of global, moderate and severe acute malnutrition) using,
#' whenever possible, standard and well-tested indicators and question sets.
#'
#' * Data analysis performed using modern computer-intensive methods to allow
#' * Data analysis performed using modern computer-intensive methods to allow
#' estimates of indicator levels to be made with useful precision using a small
#' sample size.
#'
Expand Down
24 changes: 19 additions & 5 deletions R/05-merge_op.R
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,25 @@
#' @author Ernest Guevarra
#'
#' @examples
#' \dontrun{
#' test <- merge_op(
#' x = classicEstimates, y = probitEstimates
#' )
#' }
#' indicators <- c(
#' "demo", "anthro", "food", "hunger", "adl", "disability",
#' "mental", "dementia", "health", "oedema", "screening", "income",
#' "wash", "visual", "misc"
#' )
#'
#' classicIndicators <- indicators[indicators != "anthro"]
#'
#' ## Bootstrap classic
#' classicEstimates <- estimate_classic(
#' x = indicators.ALL, w = testPSU,
#' indicators = classicIndicators, replicates = 9
#' )
#'
#' probitEstimates <- estimate_probit(
#' x = indicators.ALL, w = testPSU, replicates = 9
#' )
#'
#' merge_op(x = classicEstimates, y = probitEstimates)
#'
#' @export
#'
Expand Down
9 changes: 1 addition & 8 deletions R/utils.R
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,7 @@
#'
#' @returns A one-dimensional table with specified values
#'
#' @author Mark Myatt
#'
#' @examples
#' oldr:::fullTable(testSVY$d1, values = 1:4)
#'
#' @author Mark Myatt
#'
#' @keywords internal
#'
Expand All @@ -37,9 +33,6 @@ fullTable <- function(x, values) {
#'
#' @returns A vector of variable names
#'
#' @examples
#' oldr:::get_variables()
#'
#' @keywords internal
#'

Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ citation("oldr")
#> Mark Myatt, Ernest Guevarra, Pascale Fritsch, Katja Siling (2025).
#> _oldr: An Implementation of Rapid Assessment Method for Older
#> People_. doi:10.5281/zenodo.7505731
#> <https://doi.org/10.5281/zenodo.7505731>, R package version 0.2.1,
#> <https://doi.org/10.5281/zenodo.7505731>, R package version 0.2.2,
#> <https://rapidsurveys.io/oldr/>.
#>
#> A BibTeX entry for LaTeX users is
Expand All @@ -104,7 +104,7 @@ citation("oldr")
#> title = {oldr: An Implementation of Rapid Assessment Method for Older People},
#> author = {{Mark Myatt} and {Ernest Guevarra} and {Pascale Fritsch} and {Katja Siling}},
#> year = {2025},
#> note = {R package version 0.2.1},
#> note = {R package version 0.2.2},
#> url = {https://rapidsurveys.io/oldr/},
#> doi = {10.5281/zenodo.7505731},
#> }
Expand Down
24 changes: 18 additions & 6 deletions cran-comments.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,24 @@
## RESUBMISSION

In this resubmission, I addressed the following comments from CRAN:

* It seems like you have too many spaces in your description field. Probably because linebreaks count as spaces too. Please remove unecassary ones.

- I removed spaces from previous line before a line break and then removed the last two sentences to avoid creating more line breaks

* You have examples for unexported functions. Please either omit these examples or export these functions.

- I removed all examples from unexported functions

* `\dontrun{}` should only be used if the example really cannot be executed

- I removed `\dontrun{}` from the example for the `merge_op()` function.


## Test environments
* local OS X install, R 4.4.2
* local ubuntu 22.04 install, R 4.4.2
* win-builder (devel, release and old release)
* win-builder (devel, release, and old release)
* github actions windows-latest, r: release
* github actions macOS-latest, r: release
* github actions ubuntu-22.04, r: release, devel, old release
Expand All @@ -21,8 +38,3 @@

* This is a new release.


## Reverse dependencies

This is a new release. There are no reverse/downstream dependencies.

2 changes: 1 addition & 1 deletion inst/CITATION
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ bibentry(
title = "oldr: An Implementation of Rapid Assessment Method for Older People",
author = c(person("Mark Myatt"), person("Ernest Guevarra"), person("Pascale Fritsch"), person("Katja Siling")),
year = "2025",
note = "R package version 0.2.1",
note = "R package version 0.2.2",
url = "https://rapidsurveys.io/oldr/",
doi = "10.5281/zenodo.7505731"
)
5 changes: 0 additions & 5 deletions man/fullTable.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions man/get_variables.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

24 changes: 19 additions & 5 deletions man/merge_op.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 95f0503

Please sign in to comment.