{"action":"created","issue":{"url":"https://api.github.com/repos/dib-lab/khmer/issues/1491","repository_url":"https://api.github.com/repos/dib-lab/khmer","labels_url":"https://api.github.com/repos/dib-lab/khmer/issues/1491/labels{/name}","comments_url":"https://api.github.com/repos/dib-lab/khmer/issues/1491/comments","events_url":"https://api.github.com/repos/dib-lab/khmer/issues/1491/events","html_url":"https://github.com/dib-lab/khmer/pull/1491","id":184842284,"number":1491,"title":"expose khmer.Read to python and use ReadParser in scripts","user":{"login":"betatim","id":1448859,"avatar_url":"https://avatars.githubusercontent.com/u/1448859?v=3","gravatar_id":"","url":"https://api.github.com/users/betatim","html_url":"https://github.com/betatim","followers_url":"https://api.github.com/users/betatim/followers","following_url":"https://api.github.com/users/betatim/following{/other_user}","gists_url":"https://api.github.com/users/betatim/gists{/gist_id}","starred_url":"https://api.github.com/users/betatim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/betatim/subscriptions","organizations_url":"https://api.github.com/users/betatim/orgs","repos_url":"https://api.github.com/users/betatim/repos","events_url":"https://api.github.com/users/betatim/events{/privacy}","received_events_url":"https://api.github.com/users/betatim/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":15,"created_at":"2016-10-24T13:35:47Z","updated_at":"2016-11-14T13:21:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dib-lab/khmer/pulls/1491","html_url":"https://github.com/dib-lab/khmer/pull/1491","diff_url":"https://github.com/dib-lab/khmer/pull/1491.diff","patch_url":"https://github.com/dib-lab/khmer/pull/1491.patch"},"body":"Related to #1483 \r\n\r\nAfter getting started on this I realised I wasn't sure how we wanted to use this. So far I am assuming the use case is to replace `utils.ReadBundle` as used in for example `trim-low-abund.py`.\r\n\r\nIs it really worth it? We will end up turning all the screed `Records` into `read_parser::Read`s via various unicode -> bytes stuff, to then do some computation on them. In the current setup we pass the cleaned sequence to `get_median_count` and friends. Maybe there is something I am missing.\r\n- [x] Is it mergeable?\r\n- [x] `make test` Did it pass the tests?\r\n- [x] `make clean diff-cover` If it introduces new functionality in\r\n  `scripts/` is it tested?\r\n- [x] `make format diff_pylint_report cppcheck doc pydocstyle` Is it well\r\n  formatted?\r\n- [ ] Did it change the command-line interface? Only additions are allowed\r\n  without a major version increment. Changing file formats also requires a\r\n  major version number increment.\r\n- [x] Is it documented in the `ChangeLog`?\r\n  http://en.wikipedia.org/wiki/Changelog#Format\r\n- [x] Was a spellchecker run on the source code and documentation after\r\n  changes were made?\r\n- [x] Do the changes respect streaming IO? (Are they\r\n  tested for streaming IO?)\r\n- [x] Is the Copyright year up to date?\r\n\r\nReadBundles group several Reads together.\r\n"},"comment":{"url":"https://api.github.com/repos/dib-lab/khmer/issues/comments/260332754","html_url":"https://github.com/dib-lab/khmer/pull/1491#issuecomment-260332754","issue_url":"https://api.github.com/repos/dib-lab/khmer/issues/1491","id":260332754,"user":{"login":"betatim","id":1448859,"avatar_url":"https://avatars.githubusercontent.com/u/1448859?v=3","gravatar_id":"","url":"https://api.github.com/users/betatim","html_url":"https://github.com/betatim","followers_url":"https://api.github.com/users/betatim/followers","following_url":"https://api.github.com/users/betatim/following{/other_user}","gists_url":"https://api.github.com/users/betatim/gists{/gist_id}","starred_url":"https://api.github.com/users/betatim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/betatim/subscriptions","organizations_url":"https://api.github.com/users/betatim/orgs","repos_url":"https://api.github.com/users/betatim/repos","events_url":"https://api.github.com/users/betatim/events{/privacy}","received_events_url":"https://api.github.com/users/betatim/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T13:21:37Z","updated_at":"2016-11-14T13:21:37Z","body":"Ready for review! @luizirber, @camillescott, @standage , or @ctb"}}
{"actor":{"display_login":"betatim"}}