{"action":"created","issue":{"url":"https://api.github.com/repos/datamade/dedupe-api/issues/200","labels_url":"https://api.github.com/repos/datamade/dedupe-api/issues/200/labels{/name}","comments_url":"https://api.github.com/repos/datamade/dedupe-api/issues/200/comments","events_url":"https://api.github.com/repos/datamade/dedupe-api/issues/200/events","html_url":"https://github.com/datamade/dedupe-api/issues/200","id":74003599,"number":200,"title":"Code cleanup","user":{"login":"evz","id":551491,"avatar_url":"https://avatars.githubusercontent.com/u/551491?v=3","gravatar_id":"","url":"https://api.github.com/users/evz","html_url":"https://github.com/evz","followers_url":"https://api.github.com/users/evz/followers","following_url":"https://api.github.com/users/evz/following{/other_user}","gists_url":"https://api.github.com/users/evz/gists{/gist_id}","starred_url":"https://api.github.com/users/evz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/evz/subscriptions","organizations_url":"https://api.github.com/users/evz/orgs","repos_url":"https://api.github.com/users/evz/repos","events_url":"https://api.github.com/users/evz/events{/privacy}","received_events_url":"https://api.github.com/users/evz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-07T14:21:22Z","updated_at":"2015-05-07T14:23:02Z","closed_at":null,"body":"Now that we are basically done with milestone 0.2, I think it might be nice to do a bit of work reorganizing and cleaning up this code. Among the things I'd like to accomplish:\r\n\r\n- [ ] Make sense of ``review.py``, and ``matching.py`` in light of the recent step reversal. This might involve just making a module for each review step (or collapsing it into one, whichever seems the most sane)\r\n- [ ] Breakdown large view functions in ``admin.py``, ``review.py``, ``matching.py``, and ``training.py`` into smaller, easier to comprehend functions\r\n- [ ] Do #189 and #190 \r\n- [ ] Breakdown functions in ``db_functions.py`` and ``helpers.py`` into modules that map better to the steps in the canonicalization process. \r\n- [ ] Breakdown functions in ``delayed_tasks.py`` so that they are easier to reason about."},"comment":{"url":"https://api.github.com/repos/datamade/dedupe-api/issues/comments/99884552","html_url":"https://github.com/datamade/dedupe-api/issues/200#issuecomment-99884552","issue_url":"https://api.github.com/repos/datamade/dedupe-api/issues/200","id":99884552,"user":{"login":"fgregg","id":536941,"avatar_url":"https://avatars.githubusercontent.com/u/536941?v=3","gravatar_id":"","url":"https://api.github.com/users/fgregg","html_url":"https://github.com/fgregg","followers_url":"https://api.github.com/users/fgregg/followers","following_url":"https://api.github.com/users/fgregg/following{/other_user}","gists_url":"https://api.github.com/users/fgregg/gists{/gist_id}","starred_url":"https://api.github.com/users/fgregg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fgregg/subscriptions","organizations_url":"https://api.github.com/users/fgregg/orgs","repos_url":"https://api.github.com/users/fgregg/repos","events_url":"https://api.github.com/users/fgregg/events{/privacy}","received_events_url":"https://api.github.com/users/fgregg/received_events","type":"User","site_admin":false},"created_at":"2015-05-07T14:23:02Z","updated_at":"2015-05-07T14:23:02Z","body":"I think moving things from delayed_tasks might also make since. That tasks are delayed is not the most compelling similarity."}}