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

[hotfix] Correct Senate classes in constants.py #1430

Closed
Tracked by #113
lbeaufort opened this issue Oct 31, 2017 · 1 comment
Closed
Tracked by #113

[hotfix] Correct Senate classes in constants.py #1430

lbeaufort opened this issue Oct 31, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@lbeaufort
Copy link
Member

lbeaufort commented Oct 31, 2017

Per @PaulClark2:
"I did find a mistake in the Senate classes but not with Texas. Texas was correctly in Class 1. Here's the correct class information. I've corrected this in FECP. We'll need to correct it constants.py."

'1': ['AZ', 'CA', 'CT', 'DE', 'FL', 'HI', 'IN', 'ME', 'MD', 'MA', 'MI', 'MN', 'MS', 'MO', 'MT', 'NE', 'NV', 'NJ', 'NM', 'NY', 'ND', 'OH', 'PA', 'RI', 'TN', 'TX', 'UT', 'VT', 'VA', 'WA', 'WV', 'WI', 'WY'],

'2': ['AL', 'AK', 'AR', 'CO', 'DE', 'GA', 'ID', 'IL', 'IA', 'KS', 'KY', 'LA', 'ME', 'MA', 'MI', 'MN', 'MS', 'MT', 'NE', 'NH', 'NJ', 'NM', 'NC', 'OK', 'OR', 'RI', 'SC', 'SD', 'TN', 'TX', 'VA', 'WV', 'WY'],

'3': ['AL', 'AK', 'AZ', 'AR', 'CA', 'CO', 'CT', 'FL', 'GA', 'HI', 'ID', 'IL', 'IN', 'IA', 'KS', 'KY', 'LA', 'MD', 'MO', 'NV', 'NH', 'NY', 'NC', 'ND', 'OH', 'OK', 'OR', 'PA', 'SC', 'SD', 'UT', 'VT', 'WA', 'WI']

@PaulClark2
Copy link
Contributor

I'd like to make this a hotfix instead of waiting for our next release. A number of states in Classes 2 and 3 where wrong in FECP. I've corrected the data in FECP.

@lbeaufort lbeaufort self-assigned this Oct 31, 2017
@lbeaufort lbeaufort added the Bug label Oct 31, 2017
@lbeaufort lbeaufort added this to the Sprint 4.3 milestone Nov 1, 2017
@lbeaufort lbeaufort changed the title Correct Senate classes in constants.py [hotfix] Correct Senate classes in constants.py Nov 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants