{"action":"created","comment":{"url":"https://api.github.com/repos/dask/dask/pulls/comments/232086609","pull_request_review_id":173205251,"id":232086609,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIzMjA4NjYwOQ==","diff_hunk":"@@ -66,31 +66,32 @@ key-value pairs required to eventually compute the desired values, for example\n    }\n \n The name of an ``Array`` object can be found in the ``name`` attribute.  One\n-can get a nested list of keys with the ``.__dask_keys__()`` method.  One can\n-flatten down this list with ``dask.array.core.flatten()``; this is sometimes\n+can get a nested list of keys with the ``.__dask_keys__()`` method.  Additionally, \n+one can flatten down this list with ``dask.array.core.flatten()``. This is sometimes\n useful when building new dictionaries.\n \n Chunks\n ------\n \n-We also store the size of each block along each axis.  This is a tuple of\n-tuples such that the length of the outer tuple is equal to the dimension and\n-the lengths of the inner tuples are equal to the number of blocks along each\n-dimension.  In the example illustrated above this value is as follows::\n+We also store the size of each block along each axis.  This is composed of \n+a tuple of tuples such that the length of the outer tuple is equal to the \n+number of dimensions of the array, and the lengths of the inner tuples are \n+equal to the number of blocks along each dimension.  In the example illustrated ","path":"docs/source/array-design.rst","position":57,"original_position":57,"commit_id":"87a4a04128d378f8ae89daba24ef3c7ef33b61bb","original_commit_id":"866ec4692e37c565071b0d80573987b748958da8","user":{"login":"farrajota","id":15182939,"node_id":"MDQ6VXNlcjE1MTgyOTM5","avatar_url":"https://avatars2.githubusercontent.com/u/15182939?v=4","gravatar_id":"","url":"https://api.github.com/users/farrajota","html_url":"https://github.com/farrajota","followers_url":"https://api.github.com/users/farrajota/followers","following_url":"https://api.github.com/users/farrajota/following{/other_user}","gists_url":"https://api.github.com/users/farrajota/gists{/gist_id}","starred_url":"https://api.github.com/users/farrajota/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farrajota/subscriptions","organizations_url":"https://api.github.com/users/farrajota/orgs","repos_url":"https://api.github.com/users/farrajota/repos","events_url":"https://api.github.com/users/farrajota/events{/privacy}","received_events_url":"https://api.github.com/users/farrajota/received_events","type":"User","site_admin":false},"body":"To avoid inducing any confusion, I would keep the use of `blocks` instead of `chunks` in this section.","created_at":"2018-11-08T22:37:34Z","updated_at":"2018-11-08T22:37:34Z","html_url":"https://github.com/dask/dask/pull/4180#discussion_r232086609","pull_request_url":"https://api.github.com/repos/dask/dask/pulls/4180","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/dask/dask/pulls/comments/232086609"},"html":{"href":"https://github.com/dask/dask/pull/4180#discussion_r232086609"},"pull_request":{"href":"https://api.github.com/repos/dask/dask/pulls/4180"}},"in_reply_to_id":231341339},"pull_request":{"url":"https://api.github.com/repos/dask/dask/pulls/4180","id":228778096,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4Nzc4MDk2","html_url":"https://github.com/dask/dask/pull/4180","diff_url":"https://github.com/dask/dask/pull/4180.diff","patch_url":"https://github.com/dask/dask/pull/4180.patch","issue_url":"https://api.github.com/repos/dask/dask/issues/4180","number":4180,"state":"open","locked":false,"title":"Docs: Update array-design.rst [skip ci]","user":{"login":"farrajota","id":15182939,"node_id":"MDQ6VXNlcjE1MTgyOTM5","avatar_url":"https://avatars2.githubusercontent.com/u/15182939?v=4","gravatar_id":"","url":"https://api.github.com/users/farrajota","html_url":"https://github.com/farrajota","followers_url":"https://api.github.com/users/farrajota/followers","following_url":"https://api.github.com/users/farrajota/following{/other_user}","gists_url":"https://api.github.com/users/farrajota/gists{/gist_id}","starred_url":"https://api.github.com/users/farrajota/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farrajota/subscriptions","organizations_url":"https://api.github.com/users/farrajota/orgs","repos_url":"https://api.github.com/users/farrajota/repos","events_url":"https://api.github.com/users/farrajota/events{/privacy}","received_events_url":"https://api.github.com/users/farrajota/received_events","type":"User","site_admin":false},"body":"This PR updates `array.design.rst` with minor corrections to punctuation and some wording improvements. \r\n\r\nAlthough most of the changes are trivial, one phrase in particular was confusing / hard to understand, and I fear my modification is not correct. Hence, I need someone more knowledgeable about this topic to provide some help. The phrase in particular is the following (second phrase in [Chunks](http://docs.dask.org/en/latest/array-design.html#chunks) section):\r\n\r\n### original\r\n\r\nWe also store the size of each block along each axis. This is a tuple of tuples such that the length of the outer tuple is equal to the **dimension** and the lengths of the inner tuples are equal to the number of blocks along each dimension.\r\n\r\n### modified\r\n\r\nWe also store the size of each block along each axis.  This is composed of a tuple of tuples such that the length of the outer tuple is equal to the **number of dimensions of the array**, and the lengths of the inner tuples are equal to the number of blocks along each dimension.\r\n","created_at":"2018-11-06T17:43:10Z","updated_at":"2018-11-08T22:37:34Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e9ccc942e0ad00f78a45354da1e651a5866b2ed8","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/dask/dask/pulls/4180/commits","review_comments_url":"https://api.github.com/repos/dask/dask/pulls/4180/comments","review_comment_url":"https://api.github.com/repos/dask/dask/pulls/comments{/number}","comments_url":"https://api.github.com/repos/dask/dask/issues/4180/comments","statuses_url":"https://api.github.com/repos/dask/dask/statuses/87a4a04128d378f8ae89daba24ef3c7ef33b61bb","head":{"label":"farrajota:patch-26","ref":"patch-26","sha":"87a4a04128d378f8ae89daba24ef3c7ef33b61bb","user":{"login":"farrajota","id":15182939,"node_id":"MDQ6VXNlcjE1MTgyOTM5","avatar_url":"https://avatars2.githubusercontent.com/u/15182939?v=4","gravatar_id":"","url":"https://api.github.com/users/farrajota","html_url":"https://github.com/farrajota","followers_url":"https://api.github.com/users/farrajota/followers","following_url":"https://api.github.com/users/farrajota/following{/other_user}","gists_url":"https://api.github.com/users/farrajota/gists{/gist_id}","starred_url":"https://api.github.com/users/farrajota/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farrajota/subscriptions","organizations_url":"https://api.github.com/users/farrajota/orgs","repos_url":"https://api.github.com/users/farrajota/repos","events_url":"https://api.github.com/users/farrajota/events{/privacy}","received_events_url":"https://api.github.com/users/farrajota/received_events","type":"User","site_admin":false},"repo":{"id":149597746,"node_id":"MDEwOlJlcG9zaXRvcnkxNDk1OTc3NDY=","name":"dask","full_name":"farrajota/dask","private":false,"owner":{"login":"farrajota","id":15182939,"node_id":"MDQ6VXNlcjE1MTgyOTM5","avatar_url":"https://avatars2.githubusercontent.com/u/15182939?v=4","gravatar_id":"","url":"https://api.github.com/users/farrajota","html_url":"https://github.com/farrajota","followers_url":"https://api.github.com/users/farrajota/followers","following_url":"https://api.github.com/users/farrajota/following{/other_user}","gists_url":"https://api.github.com/users/farrajota/gists{/gist_id}","starred_url":"https://api.github.com/users/farrajota/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/farrajota/subscriptions","organizations_url":"https://api.github.com/users/farrajota/orgs","repos_url":"https://api.github.com/users/farrajota/repos","events_url":"https://api.github.com/users/farrajota/events{/privacy}","received_events_url":"https://api.github.com/users/farrajota/received_events","type":"User","site_admin":false},"html_url":"https://github.com/farrajota/dask","description":"Parallel computing with task scheduling","fork":true,"url":"https://api.github.com/repos/farrajota/dask","forks_url":"https://api.github.com/repos/farrajota/dask/forks","keys_url":"https://api.github.com/repos/farrajota/dask/keys{/key_id}","collaborators_url":"https://api.github.com/repos/farrajota/dask/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/farrajota/dask/teams","hooks_url":"https://api.github.com/repos/farrajota/dask/hooks","issue_events_url":"https://api.github.com/repos/farrajota/dask/issues/events{/number}","events_url":"https://api.github.com/repos/farrajota/dask/events","assignees_url":"https://api.github.com/repos/farrajota/dask/assignees{/user}","branches_url":"https://api.github.com/repos/farrajota/dask/branches{/branch}","tags_url":"https://api.github.com/repos/farrajota/dask/tags","blobs_url":"https://api.github.com/repos/farrajota/dask/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/farrajota/dask/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/farrajota/dask/git/refs{/sha}","trees_url":"https://api.github.com/repos/farrajota/dask/git/trees{/sha}","statuses_url":"https://api.github.com/repos/farrajota/dask/statuses/{sha}","languages_url":"https://api.github.com/repos/farrajota/dask/languages","stargazers_url":"https://api.github.com/repos/farrajota/dask/stargazers","contributors_url":"https://api.github.com/repos/farrajota/dask/contributors","subscribers_url":"https://api.github.com/repos/farrajota/dask/subscribers","subscription_url":"https://api.github.com/repos/farrajota/dask/subscription","commits_url":"https://api.github.com/repos/farrajota/dask/commits{/sha}","git_commits_url":"https://api.github.com/repos/farrajota/dask/git/commits{/sha}","comments_url":"https://api.github.com/repos/farrajota/dask/comments{/number}","issue_comment_url":"https://api.github.com/repos/farrajota/dask/issues/comments{/number}","contents_url":"https://api.github.com/repos/farrajota/dask/contents/{+path}","compare_url":"https://api.github.com/repos/farrajota/dask/compare/{base}...{head}","merges_url":"https://api.github.com/repos/farrajota/dask/merges","archive_url":"https://api.github.com/repos/farrajota/dask/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/farrajota/dask/downloads","issues_url":"https://api.github.com/repos/farrajota/dask/issues{/number}","pulls_url":"https://api.github.com/repos/farrajota/dask/pulls{/number}","milestones_url":"https://api.github.com/repos/farrajota/dask/milestones{/number}","notifications_url":"https://api.github.com/repos/farrajota/dask/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/farrajota/dask/labels{/name}","releases_url":"https://api.github.com/repos/farrajota/dask/releases{/id}","deployments_url":"https://api.github.com/repos/farrajota/dask/deployments","created_at":"2018-09-20T11:19:32Z","updated_at":"2018-09-20T11:19:36Z","pushed_at":"2018-11-07T22:01:36Z","git_url":"git://github.com/farrajota/dask.git","ssh_url":"git@github.com:farrajota/dask.git","clone_url":"https://github.com/farrajota/dask.git","svn_url":"https://github.com/farrajota/dask","homepage":"http://dask.pydata.org","size":18453,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"dask:master","ref":"master","sha":"c966e078ae4dc195a24434498ca31e145bb6bc62","user":{"login":"dask","id":17131925,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE3MTMxOTI1","avatar_url":"https://avatars1.githubusercontent.com/u/17131925?v=4","gravatar_id":"","url":"https://api.github.com/users/dask","html_url":"https://github.com/dask","followers_url":"https://api.github.com/users/dask/followers","following_url":"https://api.github.com/users/dask/following{/other_user}","gists_url":"https://api.github.com/users/dask/gists{/gist_id}","starred_url":"https://api.github.com/users/dask/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dask/subscriptions","organizations_url":"https://api.github.com/users/dask/orgs","repos_url":"https://api.github.com/users/dask/repos","events_url":"https://api.github.com/users/dask/events{/privacy}","received_events_url":"https://api.github.com/users/dask/received_events","type":"Organization","site_admin":false},"repo":{"id":28782747,"node_id":"MDEwOlJlcG9zaXRvcnkyODc4Mjc0Nw==","name":"dask","full_name":"dask/dask","private":false,"owner":{"login":"dask","id":17131925,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE3MTMxOTI1","avatar_url":"https://avatars1.githubusercontent.com/u/17131925?v=4","gravatar_id":"","url":"https://api.github.com/users/dask","html_url":"https://github.com/dask","followers_url":"https://api.github.com/users/dask/followers","following_url":"https://api.github.com/users/dask/following{/other_user}","gists_url":"https://api.github.com/users/dask/gists{/gist_id}","starred_url":"https://api.github.com/users/dask/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dask/subscriptions","organizations_url":"https://api.github.com/users/dask/orgs","repos_url":"https://api.github.com/users/dask/repos","events_url":"https://api.github.com/users/dask/events{/privacy}","received_events_url":"https://api.github.com/users/dask/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/dask/dask","description":"Parallel computing with task scheduling","fork":false,"url":"https://api.github.com/repos/dask/dask","forks_url":"https://api.github.com/repos/dask/dask/forks","keys_url":"https://api.github.com/repos/dask/dask/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dask/dask/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dask/dask/teams","hooks_url":"https://api.github.com/repos/dask/dask/hooks","issue_events_url":"https://api.github.com/repos/dask/dask/issues/events{/number}","events_url":"https://api.github.com/repos/dask/dask/events","assignees_url":"https://api.github.com/repos/dask/dask/assignees{/user}","branches_url":"https://api.github.com/repos/dask/dask/branches{/branch}","tags_url":"https://api.github.com/repos/dask/dask/tags","blobs_url":"https://api.github.com/repos/dask/dask/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dask/dask/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dask/dask/git/refs{/sha}","trees_url":"https://api.github.com/repos/dask/dask/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dask/dask/statuses/{sha}","languages_url":"https://api.github.com/repos/dask/dask/languages","stargazers_url":"https://api.github.com/repos/dask/dask/stargazers","contributors_url":"https://api.github.com/repos/dask/dask/contributors","subscribers_url":"https://api.github.com/repos/dask/dask/subscribers","subscription_url":"https://api.github.com/repos/dask/dask/subscription","commits_url":"https://api.github.com/repos/dask/dask/commits{/sha}","git_commits_url":"https://api.github.com/repos/dask/dask/git/commits{/sha}","comments_url":"https://api.github.com/repos/dask/dask/comments{/number}","issue_comment_url":"https://api.github.com/repos/dask/dask/issues/comments{/number}","contents_url":"https://api.github.com/repos/dask/dask/contents/{+path}","compare_url":"https://api.github.com/repos/dask/dask/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dask/dask/merges","archive_url":"https://api.github.com/repos/dask/dask/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dask/dask/downloads","issues_url":"https://api.github.com/repos/dask/dask/issues{/number}","pulls_url":"https://api.github.com/repos/dask/dask/pulls{/number}","milestones_url":"https://api.github.com/repos/dask/dask/milestones{/number}","notifications_url":"https://api.github.com/repos/dask/dask/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dask/dask/labels{/name}","releases_url":"https://api.github.com/repos/dask/dask/releases{/id}","deployments_url":"https://api.github.com/repos/dask/dask/deployments","created_at":"2015-01-04T18:50:00Z","updated_at":"2018-11-08T22:06:32Z","pushed_at":"2018-11-08T20:50:52Z","git_url":"git://github.com/dask/dask.git","ssh_url":"git@github.com:dask/dask.git","clone_url":"https://github.com/dask/dask.git","svn_url":"https://github.com/dask/dask","homepage":"https://dask.org","size":18418,"stargazers_count":3670,"watchers_count":3670,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":624,"mirror_url":null,"archived":false,"open_issues_count":389,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":624,"open_issues":389,"watchers":3670,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/dask/dask/pulls/4180"},"html":{"href":"https://github.com/dask/dask/pull/4180"},"issue":{"href":"https://api.github.com/repos/dask/dask/issues/4180"},"comments":{"href":"https://api.github.com/repos/dask/dask/issues/4180/comments"},"review_comments":{"href":"https://api.github.com/repos/dask/dask/pulls/4180/comments"},"review_comment":{"href":"https://api.github.com/repos/dask/dask/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/dask/dask/pulls/4180/commits"},"statuses":{"href":"https://api.github.com/repos/dask/dask/statuses/87a4a04128d378f8ae89daba24ef3c7ef33b61bb"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"farrajota"}}