valorumdata / cmdc-tools

3 stars 4 forks source link

HHS data #94

Closed cc7768 closed 4 years ago

cc7768 commented 4 years ago

We will need to add a new provider (HHS) before we merge this PR -- I've added it to the 002_covid_data file but I have not incorporated it into the database.

sglyon commented 4 years ago

@cc7768 is this ready for another review?

sglyon commented 4 years ago

Looks like tests are failing because of the sql:

I get this when I try pulling and starting the docker image locally


/usr/local/bin/docker-entrypoint.sh: running /docker-entrypoint-initdb.d/026_hhs_tables.sql
DROP TABLE
psql:/docker-entrypoint-initdb.d/026_hhs_tables.sql:1: NOTICE:  table "hhs_covid" does not exist, skipping
CREATE TABLE
COMMENT
2020-08-07 16:27:12.235 UTC [306] ERROR:  relation "dt_idx" already exists
2020-08-07 16:27:12.235 UTC [306] STATEMENT:  CREATE INDEX dt_idx ON data.hhs_covid (dt);
psql:/docker-entrypoint-initdb.d/026_hhs_tables.sql:14: ERROR:  relation "dt_idx" already exists

EDIT

Just changed this, hopefully it fixes things :crossed_fingers: :

sglyon commented 4 years ago

Lot's of scraper failures

are these fixed anywhere else?