Closed RichardMN closed 2 years ago
This PR has been flagged as stale due to lack of activity
Commented about it in https://github.com/epiforecasts/covidregionaldata/issues/430#issuecomment-1030144617. Briefly, it doesn't look like the data source will be changing in this case. It was just the result of a confusingly worded announcement. If @RichardMN agrees, we can then close this PR.
As @Bisaloo has pointed out in the #430 conversation my PR had been based on a misunderstanding of the upstream data and isn't necessary.
This adjusts the
download
andclean_level_2
for the USA data so that it will calculate the number of years (!) of data to download and then download each separate yearly file from the @nytimes repo.I suspect that the filename and directory structure may change before we get much beyond 2025 but this is "future-proofed" in some sense.
One of the columns appears to have been renamed in the level 2 data upstream but I think this is fixed too.