Skip to content
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.

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

Rename gender column in output data to sex #79

Merged
merged 5 commits into from
Feb 22, 2024
Merged

Conversation

joshwlambert
Copy link
Member

@joshwlambert joshwlambert commented Feb 21, 2024

This PR addresses a comment from the package review PR #73 on renaming the gender column output by sim_linelist(), sim_contacts() and sim_outbreak() to sex. The functions that assign and use this column (.sim_internal() and .sample_names()) are updated, as a are tests (snapshots) and vignettes.

The NEWS.md item on renaming cnt_age and cnt_gender from v0.1.0 to age and gender has been expanded to add renaming gender to sex across all simulation functions.

Test data (testtthat/testdata/) has also been updated as these now include the new column names.

@joshwlambert joshwlambert merged commit 3c98336 into main Feb 22, 2024
8 checks passed
@joshwlambert joshwlambert deleted the rename_gender branch February 22, 2024 11:13
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.

1 participant