Skip to content

Conversation

@nmdefries
Copy link
Contributor

@nmdefries nmdefries commented Jun 22, 2022

Description

Define indicators that are not used in all cuts, mostly demographic variables. Set up themed table structure.

Indicator names are grouped by the theme (topic) they belong to. After generating estimates for all indicators sharing a region and groupby variables, indicators in each theme group are collected and saved again. Non theme-grouped files are named with "all_indicators" and themed files names contain "theme_", like "theme_vaccines".

Most important changes to review here are in contingency_aggregate.R and contingency_write.R.

Changelog

  • DESCRIPTION
  • NAMESPACE
  • contingency_aggregate.R
  • contingency_indicators.R
  • contingency_variables.R
  • contingency_write.R
  • *.Rd files for various functions.

@nmdefries nmdefries marked this pull request as ready for review June 23, 2022 22:40
@nmdefries nmdefries requested a review from capnrefsmmat June 23, 2022 22:40
@nmdefries nmdefries requested a review from capnrefsmmat June 28, 2022 19:50
Copy link
Contributor

@capnrefsmmat capnrefsmmat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found a few coding typos


if ("B6" %in% names(input_data)) {
input_data$t_unusual_symptom_hospital <- input_data$B6 == 1
input_data$t_unusual_symptom_hospital_tried <- input_data$B6 == 1 | input_data$B6 == 3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completely forgot B6 existed

@nmdefries nmdefries requested a review from capnrefsmmat July 11, 2022 23:06
Copy link
Contributor

@capnrefsmmat capnrefsmmat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now

…hreshold

Lower contingency table threshold and round sample sizes
@nmdefries nmdefries merged commit d957a3d into ndefries/archival-new-inds Jul 15, 2022
@nmdefries nmdefries deleted the ndefries/theme-tables branch July 15, 2022 00:07
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.

3 participants