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

Simplify convictions_data/statute.py #3

Open
ghing opened this issue Aug 2, 2014 · 2 comments
Open

Simplify convictions_data/statute.py #3

ghing opened this issue Aug 2, 2014 · 2 comments
Assignees
Labels

Comments

@ghing
Copy link
Contributor

ghing commented Aug 2, 2014

This is related to sc3/cook-convictions#35. Now that sc3/python-iucr#3 is implemented, allowing lookup of ILCS references that include subsection bits, see if we can simplify/clean up the code in convictions_data/statute.py and code more of the statutes in our dataset.

Ulitmately, if the parsing code seems simple enough or it's easy to abstract it to different ILCS formats, we could move this to the iucr package to close sc3/python-iucr#8, though that still seems like a low priority to me.

@ghing ghing added the ready label Aug 2, 2014
@ghing ghing changed the title Update ILCS to IUCR code. Update ILCS parsing and IUCR lookup Aug 2, 2014
@bepetersn bepetersn changed the title Update ILCS parsing and IUCR lookup Simplify convictions_data/statute.py Aug 11, 2014
@bepetersn
Copy link
Member

Some of the functionality currently in statute.py should be pushed into the ilcs package. See the issue from that repo: sc3/python-ilcs#2

@bepetersn bepetersn added backlog and removed ready labels Aug 11, 2014
@bepetersn
Copy link
Member

Also related to this, iterate over the ILCS and ILRS codes we currently aren't matching to try to improve our regexes / parsing: #7.

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