Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support for 2018 acs data #82

Closed
wants to merge 2 commits into from
Closed

Conversation

mr-fuller
Copy link
Contributor

No description provided.

@MaxGhenis MaxGhenis mentioned this pull request Jan 3, 2020
@MaxGhenis
Copy link
Contributor

I think these two changes to https://github.com/datamade/census/blob/master/census/tests/test_census.py should address the build failures:

  1. Specify 2017 in the test that checks for La Canada Flintridge with 2017 data.

Replace this line:

self._client.acs5.get('NAME', geo=geo)[0]['NAME'],

with:

            self._client.acs5.get('NAME', geo=geo, year=2017)[0]['NAME'],
  1. Update the following instances of 115th Congress to 116th Congress:

'Congressional District 6 (115th Congress), Maryland'),
('state_congressional_district',
'Congressional District 6 (115th Congress), Maryland'),
('state_legislative_district_upper',

('state_congressional_district',
'Congressional District 6 (115th Congress), Maryland'),

@fgregg
Copy link
Member

fgregg commented Jan 6, 2020

agree @MaxGhenis

@fgregg
Copy link
Member

fgregg commented Apr 23, 2020

closed by #89

@fgregg fgregg closed this Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants