Skip to content

Add Composite Action For Generating Weekly Hub Visualization Data#151

Open
O957 wants to merge 2 commits intomainfrom
upx3_add_composite_action_for_visualization_data_generation
Open

Add Composite Action For Generating Weekly Hub Visualization Data#151
O957 wants to merge 2 commits intomainfrom
upx3_add_composite_action_for_visualization_data_generation

Conversation

@O957
Copy link
Collaborator

@O957 O957 commented Feb 17, 2026

This PR (from last week) adds a composite action to hubhelpr for weekly hub visualization data.

Previously, the R script for generating the three data files lived inline in the cfa-forecast-hub-reports workflow generate-visualization-data-v2.yaml. Now this workflow calls the hubhelpr action, using a few inputs (disease, token, hub path, reference date).

@O957 O957 self-assigned this Feb 17, 2026
@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 9.66%. Comparing base (60bd013) to head (fb4a24a).

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #151   +/-   ##
=====================================
  Coverage   9.66%   9.66%           
=====================================
  Files         12      12           
  Lines       1180    1180           
=====================================
  Hits         114     114           
  Misses      1066    1066           

☔ View full report in Codecov by Sentry.
📢 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.

pull_nhsn = TRUE,
pull_nssp = ref_date >= as.Date("2025-06-21")
)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please also add write_webtext()

hub_reports_path = hub_reports_path,
disease = disease,
pull_nhsn = TRUE,
pull_nssp = ref_date >= as.Date("2025-06-21")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
pull_nssp = ref_date >= as.Date("2025-06-21")
pull_nssp = TRUE

reference_date:
description: "Reference date for the forecast (YYYY-MM-DD, must be a Saturday). Defaults to the last day of the current MMWR epiweek."
required: false
default: "latest"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Once you add write_webtext(), you will also need targets input

} else {
ref_date <- as.Date(input_date)
}
cat("Using reference date:", as.character(ref_date), "\n")
Copy link
Collaborator

Choose a reason for hiding this comment

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

isn't echo better here?

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

Comments