catalyst-cooperative / pudl

The Public Utility Data Liberation Project provides analysis-ready energy system data to climate advocates, researchers, policymakers, and journalists.
https://catalyst.coop/pudl
MIT License
468 stars 107 forks source link

Incorrect county FIPS code for Bedford, VA #3531

Open gschivley opened 5 months ago

gschivley commented 5 months ago

Describe the bug

The addfips package is labeling Bedford, VA as '51515', which is the code for Bedford City. It should actually be '51019' (Bedford County). See their list of FIPS codes.

Bug Severity

How badly is this bug affecting you? Medium: I was able to identify and fix the bug in my own workflow but it might affect other people.

To Reproduce

I found the error in the core_eia861__yearly_service_territory table. Census population files do not have the FIPS code 51515.

e-belfer commented 5 months ago

https://github.com/fitnr/addfips/issues/8 It looks like this particular issue was flagged by @TrentonBush almost a year ago, and addressed in the underlying package but not released. So perhaps we just need to bug them to cut a new release.

e-belfer commented 2 months ago

Update just got pushed! Should be a simple matter of updating dependencies, I'll throw this issue into this sprint.