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

add us local councils #205

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukemckinstry
Copy link

this PR adds OCD-IDs for local councils in the Unites States

@lukemckinstry
Copy link
Author

lukemckinstry commented Nov 23, 2020

Hi, @jpmckinney @azuser is it possible to get this PR reviewed and merged? Thanks!

@jpmckinney
Copy link
Member

Can you summarize what you changed? Makes it easier to review.

@lukemckinstry
Copy link
Author

yes, absolutely.

Summary of changes: This PR adds adds council district records for 38 localities in the United States. It also adds districts to 3 localities with incomplete sets of council district records (Syracuse, Burlington, Mesquite). The 26 localities in this PR from California have recently transitioned from at-large to district representation in their local councils so these are newly created districts. The remaining localities simply did not have council districts listed in the OCD-ID files.

Adds districts:

  • Anaheim, CA
  • Carlsbad, CA
  • Chula Vista, CA
  • Concord, CA
  • Corona, CA
  • Costa Mesa, CA
  • El Cajon, CA
  • Escondido, CA
  • Fontana, CA
  • Fremont, CA
  • Fullerton, CA
  • Garden Grove, CA
  • Jurupa Valley, CA
  • Menlo Park, CA
  • Murrieta, CA
  • Oceanside, CA
  • Oxnard, CA
  • Rancho Cucamonga, CA
  • Santa Clara, CA
  • Santa Maria, CA
  • Santa Rosa, CA
  • Temecula, CA
  • Torrance, CA
  • Visalia, CA
  • Vista, CA
  • West Covina, CA
  • Stamford, CT
  • Waterbury, CT
  • Davie, FL
  • Fort Myers, FL
  • Gary, IN
  • Munster, IN
  • Jefferson City, MO
  • Greenville, SC
  • San Angelo, TX
  • Seattle, WA

Supplements incomplete district set:

  • Syracuse, NY
  • Mesquite, TX
  • Burlington, VT

We verified the entries with government shapefiles collected for these localities.

Thanks!

@rsimoes
Copy link
Contributor

rsimoes commented Nov 24, 2020

@lukemckinstry Could you add these to the respective state localgov files and run the compiler?

@lukemckinstry
Copy link
Author

@lukemckinstry Could you add these to the respective state localgov files and run the compiler?

Hi @rsimoes running compiler.py for united states results in a 386k line diff for identifiers/country-us.csv. Looks like the local and country files have not been synced up in years.

@rsimoes
Copy link
Contributor

rsimoes commented Dec 3, 2020

Hi @rsimoes running compiler.py for united states results in a 386k line diff for identifiers/country-us.csv. Looks like the local and country files have not been synced up in years.

Is it possible line terminators are being changed? The country-us.csv at HEAD is 100% \r\n.

@lukemckinstry
Copy link
Author

actually seems that an extra column/comma is being added to each row in the compiler script, so that would explain why all rows are in the diff

@jpmckinney jpmckinney added the US label Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants