Skip to content

Use NHSN HRD For Population Operations - #256

Draft
O957 wants to merge 4 commits into
mainfrom
upx3_use_nhsn_hrd
Draft

Use NHSN HRD For Population Operations#256
O957 wants to merge 4 commits into
mainfrom
upx3_use_nhsn_hrd

Conversation

@O957

@O957 O957 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

This PR adds functions to pull NHSN HRD.

@O957 O957 self-assigned this Aug 21, 2026
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 40.62500% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.98%. Comparing base (84179bb) to head (ce06700).

Files with missing lines Patch % Lines
R/pull_nhsn_hrd_admissions.R 40.62% 19 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #256      +/-   ##
==========================================
- Coverage   75.51%   74.98%   -0.53%     
==========================================
  Files          37       38       +1     
  Lines        2091     2123      +32     
==========================================
+ Hits         1579     1592      +13     
- Misses        512      531      +19     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@O957
O957 marked this pull request as ready for review August 21, 2026 19:29
@O957

O957 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Need to update CDCgov/hubhelpr#242 as well.

Comment thread R/pull_nhsn_hrd_admissions.R Outdated
dplyr::select(
"weekendingdate",
"jurisdiction",
count = tidyselect::all_of(admissions_columns[["count"]]),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Aren't admissions_columns[["count"]] and admissions_columns[["rate"]] scalars? If so, you shouldn't need all_of

@O957
O957 marked this pull request as draft August 25, 2026 13:39
@dylanhmorris

dylanhmorris commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

@O957 I think we should close per my comments here: CDCgov/hubhelpr#242 (comment)

See also our previous decision not to make less-generic dataset pullers in #180 (comment)

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