Open melange396 opened 1 year ago
additionally, a number of README.md
files (located alongside indicator code in this repo) show outdated instructions, and should be updated to include the runner.
+1 for standardization!
another point of note:
the "runner" uses the archiver_from_params()
utility to automatically create an archive differ as appropriate from the job's given "params", but since it does not use the "runner", nchs_mortality
explicitly creates its own S3ArchiveDiffer
in two places (1, 2).
most indicators are set (in Cronicle) to run via the
delphi_utils.runner
wrapper:changehc
google_symptoms
claims_hosp
quidel_covidtest
dsew_community_profile
those commands take the form
python -m delphi_utils.runner delphi_INDICATORNAME
some others do not run that way:
nchs_mortality
has its module run directly (python -m delphi_nchs_mortality
)hhs_hosp
runs via a call tomake run
(which has seperate steps for the pipeline, validator, and archive)doctor_visits
mysteriously runs a script (that doesnt seem to be in github) w/ a non-standard user on a non-standard machinedetermine whether we can and should convert them all to run within the "runner" that provides validation and archiving, among other things.