-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update to use new data #10
Comments
Hi! This (.xlsx) looks like the best and most recent resource via the Illinois State Police, dated December 29, 2016. I did some quick and dirty munging, looks like there are about 75 codes in the update that aren't in the lib's data (.csv). (Probably a good idea to cq this. 😅) The statutes may take more time to reconcile because I'm not sure we can assume they're in the same format. |
@hancush Thanks! Do we have to worry about IUCRs mapping to different ILCS codes? Or is this something that we should verify? |
Yes, in some cases the ILCS code may have been updated or repealed. Luckily old IUCR codes are "retained for historical purposes" in the data. Here's the key provided by ISP (emphasis added):
(The blue/bolding is in the source .xlsx I linked above.) |
Did a full diff of the existing and updated codes. This output is all codes where the ILCS statute is different in the update than it is in the existing data. Some just get more specific (including (a)/(b), etc.) and some are entirely different statutes. I can sort thru by hand. (Makefile and comparison script plus source data also in that gist.) |
Reading back over the notes, I had some thoughts. If the primary objective of this lib is to convert ILCS statutes to IUCR codes, preserving historical code/statute pairs as we update the data is in our best interest. I propose adding two columns to the underlying data:
We could also update the API, taking a leaf from Does this seem reasonable, @ghing? |
Hannah Cushman at datamade pointed out that the state released new IUCR codes. We should update this library to reflect the new codes.
The text was updated successfully, but these errors were encountered: