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.

Sign up for GitHub

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

Jump to bottom

Update BMI variable names #98

Closed
delosh653 opened this issue Sep 29, 2022 · 0 comments · Fixed by #100
Closed

Update BMI variable names #98

delosh653 opened this issue Sep 29, 2022 · 0 comments · Fixed by #100
Labels
easy Should take two hours or less
Milestone
v2.1.0

Comments

@delosh653
Copy link
Collaborator

Update variable names for BMI to correspond to new standards, i.e.:

  • Extended BMIz -> BMIz
  • BMIz -> original BMIz
  • etc.
@delosh653 delosh653 added the easy Should take two hours or less label Sep 29, 2022
dchud added a commit to mitre/growthcleanr that referenced this issue Oct 7, 2022
@dchud
Minor code reformat via RStudio, prepping for comparison, refs carrie…
1fe6155 
…daymont#98
dchud added a commit to mitre/growthcleanr that referenced this issue Oct 7, 2022
@dchud
Updated code from David Freedman, reformatted via RStudio, refs carri…
cb4e20a 
…edaymont#98
@dchud dchud linked a pull request Oct 7, 2022 that will close this issue
Update BMI variable names #100
Merged
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 1, 2023
@dchud
Tweaking whitespace, comments to align with prior version for clarity…
3698e9b 
… of review, refs carriedaymont#98
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 1, 2023
@dchud
Tweaked ext_bmiz() function doc and vignette to note Dec 2022 update,…
d65b877 
… refs carriedaymont#98
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 1, 2023
@dchud
Updated ext_bmi() docs to enumerate possible values for sex, refs car…
36b3d8e 
…riedaymont#98
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 1, 2023
@dchud
Resolved merge conflicts in generated docs, refs carriedaymont#98
9e0ae59
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 2, 2023
@dchud
Added explicit link to NCHS report in ext_bmiz() docs, refs carrieday…
d9c9d30 
…mont#98
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 2, 2023
@dchud
Tweaked ext_bmiz() docs further for clarity, refs carriedaymont#98
efb4d80
dchud added a commit to mitre/growthcleanr that referenced this issue Feb 3, 2023
@dchud
More tweaks to text flow in utilities vignette, refs carriedaymont#98
1ca41fb
@dchud dchud added this to the v2.1.0 milestone Feb 3, 2023
dchud added a commit that referenced this issue Feb 3, 2023
@dchud
Merge pull request #100 from mitre/t98-update-bmi-varnames
2fafcdf 
Update BMI variable names and docs to match; closes #98
@dchud dchud closed this as completed in 83d5804 Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy Should take two hours or less
Projects
None yet
Milestone
v2.1.0
Development

Successfully merging a pull request may close this issue.

Update BMI variable names mitre/growthcleanr
2 participants
@dchud @delosh653