{"action":"created","issue":{"url":"https://api.github.com/repos/bccp/nbodykit/issues/420","repository_url":"https://api.github.com/repos/bccp/nbodykit","labels_url":"https://api.github.com/repos/bccp/nbodykit/issues/420/labels{/name}","comments_url":"https://api.github.com/repos/bccp/nbodykit/issues/420/comments","events_url":"https://api.github.com/repos/bccp/nbodykit/issues/420/events","html_url":"https://github.com/bccp/nbodykit/pull/420","id":263115242,"number":420,"title":"fixes #419 and remove optimized dask selection","user":{"login":"nickhand","id":2262496,"avatar_url":"https://avatars3.githubusercontent.com/u/2262496?v=4","gravatar_id":"","url":"https://api.github.com/users/nickhand","html_url":"https://github.com/nickhand","followers_url":"https://api.github.com/users/nickhand/followers","following_url":"https://api.github.com/users/nickhand/following{/other_user}","gists_url":"https://api.github.com/users/nickhand/gists{/gist_id}","starred_url":"https://api.github.com/users/nickhand/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickhand/subscriptions","organizations_url":"https://api.github.com/users/nickhand/orgs","repos_url":"https://api.github.com/users/nickhand/repos","events_url":"https://api.github.com/users/nickhand/events{/privacy}","received_events_url":"https://api.github.com/users/nickhand/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-10-05T12:56:31Z","updated_at":"2017-10-05T12:57:15Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/bccp/nbodykit/pulls/420","html_url":"https://github.com/bccp/nbodykit/pull/420","diff_url":"https://github.com/bccp/nbodykit/pull/420.diff","patch_url":"https://github.com/bccp/nbodykit/pull/420.patch"},"body":"This fixes two issues arising in v0.2.7 on NERSC: \r\n\r\n- #419: only slices that do not change the **collective** size, not local size do not need to be evaluated\r\n- dask optimized selection is buggy and can fail in a very bad way, at the time of a compute() call, breaking basically the whole code. After some more thought, I think it's best to not fiddle with the dask internals as much as possible, given that the internal API can change at any time. Also, it's a hard piece of code to debug and maintain, so it seems best to remove for both the future usability of the code and future maintenance\r\n\r\nThoughts, @rainwoodman?"},"comment":{"url":"https://api.github.com/repos/bccp/nbodykit/issues/comments/334456495","html_url":"https://github.com/bccp/nbodykit/pull/420#issuecomment-334456495","issue_url":"https://api.github.com/repos/bccp/nbodykit/issues/420","id":334456495,"user":{"login":"nickhand","id":2262496,"avatar_url":"https://avatars3.githubusercontent.com/u/2262496?v=4","gravatar_id":"","url":"https://api.github.com/users/nickhand","html_url":"https://github.com/nickhand","followers_url":"https://api.github.com/users/nickhand/followers","following_url":"https://api.github.com/users/nickhand/following{/other_user}","gists_url":"https://api.github.com/users/nickhand/gists{/gist_id}","starred_url":"https://api.github.com/users/nickhand/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickhand/subscriptions","organizations_url":"https://api.github.com/users/nickhand/orgs","repos_url":"https://api.github.com/users/nickhand/repos","events_url":"https://api.github.com/users/nickhand/events{/privacy}","received_events_url":"https://api.github.com/users/nickhand/received_events","type":"User","site_admin":false},"created_at":"2017-10-05T12:57:14Z","updated_at":"2017-10-05T12:57:14Z","author_association":"OWNER","body":"I've confirmed this fixes my recent hanging issues on NERSC, and we should push a v0.2.8 once @rainwoodman is on board."}}
{"actor":{"display_login":"nickhand"}}