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

Release covidcast-indicators 0.3.55 #1991

Merged
merged 58 commits into from
Jul 10, 2024

Conversation

rzats and others added 30 commits January 4, 2024 17:12
* pin to rocker/tidyverse:4.2
* update Dockerfile to use pak and rspm for package installation
* add workflow_dispatch button
* simplify caching (r-lib/setup-r-dependencies caches by default)
* switch to Ubuntu latest
* keep R on 4.2
first attempt at fixing broken backfill build
[Refactor] Add type hints to `geomap.py` and a small test refactor
feat: add aggregate_by_weighted_sum to geomapper
Weekday adjustment to use Clarabel instead of ECOS
* all linters now use the same configuration file
* add `make format` command to format all code
* make sure all indicators pass with the new config
* update the template files as well
* add a new darker format job to CI
* add python package caching
* update README
feat: unify linter settings across indicators and add incremental formatting
nmdefries and others added 23 commits July 1, 2024 13:25
* add patching code

* add documentation

* linter

* fix dir name

* Update get_latest_claims_name.py

* remove patch var, use only issue"

* issue -> issue_date for clarity

* fix logger

* unit test

* fix unit test

* lint

* Update doctor_visits/delphi_doctor_visits/patch.py

Co-authored-by: nmdefries <42820733+nmdefries@users.noreply.github.com>

* add download and get_latest_claims_name tests

* Update test_get_latest_claims_name.py test file name

---------

Co-authored-by: nmdefries <42820733+nmdefries@users.noreply.github.com>
Co-authored-by: David Weber <david.weber2@pm.me>
* fix doctor_visits log location (updated to be in shared system indicators log directory)
* Update doctor_visits export_dir

Co-authored-by: minhkhul <118945681+minhkhul@users.noreply.github.com>
Copy link
Contributor

@melange396 melange396 left a comment

Choose a reason for hiding this comment

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

lotsa changes... kinda scary, but i gave it a very thorough look-through 🤞

@melange396 melange396 merged commit b338a09 into prod Jul 10, 2024
2 checks passed
@melange396 melange396 deleted the release/indicators_v0.3.55_utils_v0.3.24 branch July 10, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants