search

sc3 / cookcountyjail

A Django app that tracks the population of Cook County Jail over time and summarizes trends.
http://cookcountyjail.recoveredfactory.net/api/1.0/?format=json
Other
31 stars 23 forks source link

remove comment field on charge in v2 API? #364

Closed bepetersn closed 10 years ago

bepetersn commented 10 years ago

See #363: charge comments don't map consistently to citation.

Given that we would like to normalize our database as much as possible, should we get rid of these comments on the charge object? I had a thought that if this problem came up, we could add the charge comment to the chargeshistory object, alongside the reference to the charge, since it is independent of the charge.

What do you think, @nwinklareth? @wilbertom? Others?

bepetersn commented 10 years ago

With regard to storing things alongside the reference to the charge on the chargeshistory, we could also store the raw charge string there, if we desired. @eads had always said we should keep as much of the raw data as possible, even while parsing more efficiently. But we can't keep the raw charge any more than we keep the charge comment, if we are trying to standardize charges.

bepetersn commented 10 years ago

This is no longer necessary, we've creating a new charges description.