cenpy-devs / cenpy

Explore and download data from Census APIs
Other
183 stars 44 forks source link

Update year bounds to address API changes #124 #125

Closed ronnie-llamado closed 3 years ago

ronnie-llamado commented 3 years ago

Update years bounds from [2013,2017] to [2017,2019] to reflect the API changes (Issue #124)

Year changes in ACS.products.ACS.__init__() and test_functional_products.py.

ronnie-llamado commented 3 years ago

I had the same failure on my machine, but was hoping it was just a local issue. Darn.

I think this due to another API change on TIGER map service limiting the size of requests to 32MB. I've gotten around this by choosing a much smaller metropolitan statistical area than Austin, TX. This gets around the testing for this, but has larger implications. I'll add a separate issue.

ronnie-llamado commented 3 years ago

I've added #126 to point out the failure in test_functional_products.py, seen in build Build #1464.

ljwolf commented 3 years ago

Wow, great, thanks! I'll try to get this reviewed & merged this week.