feat: update population data based on indicator data #558
Conversation
|
Preview link ready! Built with commit 39b1bc1 https://deploy-preview-558--cmu-delphi-covidcast.netlify.app |
| async function generatePopulationLookup() { | ||
| const csv = csvParse( | ||
| await fetch( | ||
| 'https://raw.githubusercontent.com/cmu-delphi/covidcast-indicators/main/_delphi_utils_python/delphi_utils/data/fips_pop.csv', |
There was a problem hiding this comment.
Are there any cross-site considerations to be made when pulling this data from github directly? Would it be better to pull the data down and package it as part of the site at build time?
There was a problem hiding this comment.
the raw data is just processed on demand. It will generate among others the CSV file that is also checked in.
There was a problem hiding this comment.
Ah so, this is only at build time? That makes sense. It'd be nice if there was a npm package that we could pull down as a first-class dependency, but until then this make sense.
| async function generatePopulationLookup() { | ||
| const csv = csvParse( | ||
| await fetch( | ||
| 'https://raw.githubusercontent.com/cmu-delphi/covidcast-indicators/main/_delphi_utils_python/delphi_utils/data/fips_pop.csv', |
There was a problem hiding this comment.
Ah so, this is only at build time? That makes sense. It'd be nice if there was a npm package that we could pull down as a first-class dependency, but until then this make sense.
closes #553
Prerequisites:
devbranchdevSummary
updates the population with the latest file as mentioned in the issue when processing the raw data: