{"action":"created","issue":{"url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/81","repository_url":"https://api.github.com/repos/zarr-developers/numcodecs","labels_url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/81/comments","events_url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/81/events","html_url":"https://github.com/zarr-developers/numcodecs/pull/81","id":321491466,"number":81,"title":"First pass for fixing blosc encode error handling.","user":{"login":"jeromekelleher","id":2664569,"avatar_url":"https://avatars2.githubusercontent.com/u/2664569?v=4","gravatar_id":"","url":"https://api.github.com/users/jeromekelleher","html_url":"https://github.com/jeromekelleher","followers_url":"https://api.github.com/users/jeromekelleher/followers","following_url":"https://api.github.com/users/jeromekelleher/following{/other_user}","gists_url":"https://api.github.com/users/jeromekelleher/gists{/gist_id}","starred_url":"https://api.github.com/users/jeromekelleher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jeromekelleher/subscriptions","organizations_url":"https://api.github.com/users/jeromekelleher/orgs","repos_url":"https://api.github.com/users/jeromekelleher/repos","events_url":"https://api.github.com/users/jeromekelleher/events{/privacy}","received_events_url":"https://api.github.com/users/jeromekelleher/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-05-09T09:12:53Z","updated_at":"2018-05-10T10:25:45Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/zarr-developers/numcodecs/pulls/81","html_url":"https://github.com/zarr-developers/numcodecs/pull/81","diff_url":"https://github.com/zarr-developers/numcodecs/pull/81.diff","patch_url":"https://github.com/zarr-developers/numcodecs/pull/81.patch"},"body":"WIP. Closes #80.\r\n\r\nSome problems:\r\n\r\n1. Provoking the error requires making a 2G array, which CI providers might not like. I guess we'll have to see how this works out as it runs.\r\n2. More seriously, there is some problem with mutexes when ``blosc.use_threads`` is true. It looks like there is some library cleanup that should be performed after an error occurs?\r\n3. Currently the error message is written to stderr/stdout, which isn't ideal. There should some mechanism for telling blosc not to write out error messages, and instead retrieve the error message so we can put it into the exception.\r\n\r\nTODO:\r\n* [ ] Unit tests and/or doctests in docstrings\r\n* [ ] ``tox -e py36`` passes locally\r\n* [ ] ``tox -e py27`` passes locally\r\n* [ ] Docstrings and API docs for any new/modified user-facing classes and functions\r\n* [ ] Changes documented in docs/release.rst\r\n* [ ] ``tox -e docs`` passes locally\r\n* [ ] AppVeyor and Travis CI passes\r\n* [ ] Test coverage to 100% (Coveralls passes)\r\n"},"comment":{"url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/comments/388015528","html_url":"https://github.com/zarr-developers/numcodecs/pull/81#issuecomment-388015528","issue_url":"https://api.github.com/repos/zarr-developers/numcodecs/issues/81","id":388015528,"user":{"login":"alimanfoo","id":703554,"avatar_url":"https://avatars1.githubusercontent.com/u/703554?v=4","gravatar_id":"","url":"https://api.github.com/users/alimanfoo","html_url":"https://github.com/alimanfoo","followers_url":"https://api.github.com/users/alimanfoo/followers","following_url":"https://api.github.com/users/alimanfoo/following{/other_user}","gists_url":"https://api.github.com/users/alimanfoo/gists{/gist_id}","starred_url":"https://api.github.com/users/alimanfoo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alimanfoo/subscriptions","organizations_url":"https://api.github.com/users/alimanfoo/orgs","repos_url":"https://api.github.com/users/alimanfoo/repos","events_url":"https://api.github.com/users/alimanfoo/events{/privacy}","received_events_url":"https://api.github.com/users/alimanfoo/received_events","type":"User","site_admin":false},"created_at":"2018-05-10T10:25:45Z","updated_at":"2018-05-10T10:25:45Z","author_association":"OWNER","body":"Thanks @jeromekelleher, SGTM."}}
{"actor":{"display_login":"alimanfoo"}}