CDCgov / cfa-config-generator

Apache License 2.0
0 stars 0 forks source link

Script to generate default `epinow2` task configurations #4

Closed amondal2 closed 1 month ago

amondal2 commented 1 month ago

Closes #3 --

This piece of the script handles generating the default set (i.e., all state-pathogen combinations) of task configurations. Future pieces will accept parameters (for now I've left them as parameters injected into the environment via os.environ, but this might change once I figure out how to integrate this with GHA) and generate only the specified tasks. Currently, this only handles one report_date.

I'll create issues separately for

but wanted to make sure the actual generation piece looks correct. @zsusswein @natemcintosh I left some clarifying questions in the comments. Thanks!

amondal2 commented 1 month ago

Thanks @natemcintosh for the suggestions! Will incorporate shortly

zsusswein commented 1 month ago

Got caught with Rt production stuff. Will try to look before I log off today

amondal2 commented 1 month ago

Added type hints and using date type throughout. @natemcintosh

amondal2 commented 1 month ago

Thanks @zsusswein -- I will address your in-line questions comments but to your main points

zsusswein commented 1 month ago

what were you thinking for a slug for the job id?

See here