{"action":"created","comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/comments/54967991","id":54967991,"diff_hunk":"@@ -2027,7 +2030,28 @@ module ChapelArray {\n     proc tail(): this._value.eltType {\n       return this[this.domain.high];\n     }\n-","path":"modules/internal/ChapelArray.chpl","position":14,"original_position":14,"commit_id":"353e252fa977a761b67ad78983091f5fb7bd9d0b","original_commit_id":"353e252fa977a761b67ad78983091f5fb7bd9d0b","user":{"login":"vasslitvinov","id":8039635,"avatar_url":"https://avatars.githubusercontent.com/u/8039635?v=3","gravatar_id":"","url":"https://api.github.com/users/vasslitvinov","html_url":"https://github.com/vasslitvinov","followers_url":"https://api.github.com/users/vasslitvinov/followers","following_url":"https://api.github.com/users/vasslitvinov/following{/other_user}","gists_url":"https://api.github.com/users/vasslitvinov/gists{/gist_id}","starred_url":"https://api.github.com/users/vasslitvinov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vasslitvinov/subscriptions","organizations_url":"https://api.github.com/users/vasslitvinov/orgs","repos_url":"https://api.github.com/users/vasslitvinov/repos","events_url":"https://api.github.com/users/vasslitvinov/events{/privacy}","received_events_url":"https://api.github.com/users/vasslitvinov/received_events","type":"User","site_admin":false},"body":"There is a comment earlier in this file that needs updating:\r\n\r\n       The current implementation reallocates the array every time the domain\r\n       is modified.  This could be improved with a size doubling/halving\r\n       strategy.\r\n","created_at":"2016-03-03T23:37:31Z","updated_at":"2016-03-03T23:37:31Z","html_url":"https://github.com/chapel-lang/chapel/pull/3380#discussion_r54967991","pull_request_url":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380","_links":{"self":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/comments/54967991"},"html":{"href":"https://github.com/chapel-lang/chapel/pull/3380#discussion_r54967991"},"pull_request":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380"}}},"pull_request":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380","id":60915653,"html_url":"https://github.com/chapel-lang/chapel/pull/3380","diff_url":"https://github.com/chapel-lang/chapel/pull/3380.diff","patch_url":"https://github.com/chapel-lang/chapel/pull/3380.patch","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/3380","number":3380,"state":"open","locked":false,"title":"Halving/Doubling style array vector operations","user":{"login":"daviditen","id":7740834,"avatar_url":"https://avatars.githubusercontent.com/u/7740834?v=3","gravatar_id":"","url":"https://api.github.com/users/daviditen","html_url":"https://github.com/daviditen","followers_url":"https://api.github.com/users/daviditen/followers","following_url":"https://api.github.com/users/daviditen/following{/other_user}","gists_url":"https://api.github.com/users/daviditen/gists{/gist_id}","starred_url":"https://api.github.com/users/daviditen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daviditen/subscriptions","organizations_url":"https://api.github.com/users/daviditen/orgs","repos_url":"https://api.github.com/users/daviditen/repos","events_url":"https://api.github.com/users/daviditen/events{/privacy}","received_events_url":"https://api.github.com/users/daviditen/received_events","type":"User","site_admin":false},"body":"Implement the array vector operations using a buffer that grows and shrinks\r\nby a factor of its current size instead of one element at a time.  Currently,\r\nthe growth factor defaults to 1.5, but it can be changed with a config\r\nparam.\r\n\r\nUpdated timeVectorArray.chpl test to include testing the remove() function.\r\n\r\nUpdate domainMaps.chpl memory tracking test .goods.  The array-as-vec work\r\nadded a new domain member to DefaultRectangular arrays.  This test tracks\r\nleaked memory from operations on different domain maps, and in some cases\r\nDefaultRectangular arrays are leaked in it.  Since the leaked arrays have\r\na new domain field, the amount leaked for each leaked array went up.","created_at":"2016-02-26T22:40:23Z","updated_at":"2016-03-03T23:37:31Z","closed_at":null,"merged_at":null,"merge_commit_sha":"42f7c74761c8c47abce43b588991b6f188aad499","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380/commits","review_comments_url":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380/comments","review_comment_url":"https://api.github.com/repos/chapel-lang/chapel/pulls/comments{/number}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/3380/comments","statuses_url":"https://api.github.com/repos/chapel-lang/chapel/statuses/353e252fa977a761b67ad78983091f5fb7bd9d0b","head":{"label":"daviditen:array-as-vec-improved","ref":"array-as-vec-improved","sha":"353e252fa977a761b67ad78983091f5fb7bd9d0b","user":{"login":"daviditen","id":7740834,"avatar_url":"https://avatars.githubusercontent.com/u/7740834?v=3","gravatar_id":"","url":"https://api.github.com/users/daviditen","html_url":"https://github.com/daviditen","followers_url":"https://api.github.com/users/daviditen/followers","following_url":"https://api.github.com/users/daviditen/following{/other_user}","gists_url":"https://api.github.com/users/daviditen/gists{/gist_id}","starred_url":"https://api.github.com/users/daviditen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daviditen/subscriptions","organizations_url":"https://api.github.com/users/daviditen/orgs","repos_url":"https://api.github.com/users/daviditen/repos","events_url":"https://api.github.com/users/daviditen/events{/privacy}","received_events_url":"https://api.github.com/users/daviditen/received_events","type":"User","site_admin":false},"repo":{"id":21909629,"name":"chapel","full_name":"daviditen/chapel","owner":{"login":"daviditen","id":7740834,"avatar_url":"https://avatars.githubusercontent.com/u/7740834?v=3","gravatar_id":"","url":"https://api.github.com/users/daviditen","html_url":"https://github.com/daviditen","followers_url":"https://api.github.com/users/daviditen/followers","following_url":"https://api.github.com/users/daviditen/following{/other_user}","gists_url":"https://api.github.com/users/daviditen/gists{/gist_id}","starred_url":"https://api.github.com/users/daviditen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daviditen/subscriptions","organizations_url":"https://api.github.com/users/daviditen/orgs","repos_url":"https://api.github.com/users/daviditen/repos","events_url":"https://api.github.com/users/daviditen/events{/privacy}","received_events_url":"https://api.github.com/users/daviditen/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/daviditen/chapel","description":"a Productive Parallel Programming Language","fork":true,"url":"https://api.github.com/repos/daviditen/chapel","forks_url":"https://api.github.com/repos/daviditen/chapel/forks","keys_url":"https://api.github.com/repos/daviditen/chapel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/daviditen/chapel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/daviditen/chapel/teams","hooks_url":"https://api.github.com/repos/daviditen/chapel/hooks","issue_events_url":"https://api.github.com/repos/daviditen/chapel/issues/events{/number}","events_url":"https://api.github.com/repos/daviditen/chapel/events","assignees_url":"https://api.github.com/repos/daviditen/chapel/assignees{/user}","branches_url":"https://api.github.com/repos/daviditen/chapel/branches{/branch}","tags_url":"https://api.github.com/repos/daviditen/chapel/tags","blobs_url":"https://api.github.com/repos/daviditen/chapel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/daviditen/chapel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/daviditen/chapel/git/refs{/sha}","trees_url":"https://api.github.com/repos/daviditen/chapel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/daviditen/chapel/statuses/{sha}","languages_url":"https://api.github.com/repos/daviditen/chapel/languages","stargazers_url":"https://api.github.com/repos/daviditen/chapel/stargazers","contributors_url":"https://api.github.com/repos/daviditen/chapel/contributors","subscribers_url":"https://api.github.com/repos/daviditen/chapel/subscribers","subscription_url":"https://api.github.com/repos/daviditen/chapel/subscription","commits_url":"https://api.github.com/repos/daviditen/chapel/commits{/sha}","git_commits_url":"https://api.github.com/repos/daviditen/chapel/git/commits{/sha}","comments_url":"https://api.github.com/repos/daviditen/chapel/comments{/number}","issue_comment_url":"https://api.github.com/repos/daviditen/chapel/issues/comments{/number}","contents_url":"https://api.github.com/repos/daviditen/chapel/contents/{+path}","compare_url":"https://api.github.com/repos/daviditen/chapel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/daviditen/chapel/merges","archive_url":"https://api.github.com/repos/daviditen/chapel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/daviditen/chapel/downloads","issues_url":"https://api.github.com/repos/daviditen/chapel/issues{/number}","pulls_url":"https://api.github.com/repos/daviditen/chapel/pulls{/number}","milestones_url":"https://api.github.com/repos/daviditen/chapel/milestones{/number}","notifications_url":"https://api.github.com/repos/daviditen/chapel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/daviditen/chapel/labels{/name}","releases_url":"https://api.github.com/repos/daviditen/chapel/releases{/id}","deployments_url":"https://api.github.com/repos/daviditen/chapel/deployments","created_at":"2014-07-16T17:06:41Z","updated_at":"2016-01-12T22:06:50Z","pushed_at":"2016-03-03T06:00:48Z","git_url":"git://github.com/daviditen/chapel.git","ssh_url":"git@github.com:daviditen/chapel.git","clone_url":"https://github.com/daviditen/chapel.git","svn_url":"https://github.com/daviditen/chapel","homepage":"http://chapel.cray.com/","size":356130,"stargazers_count":2,"watchers_count":2,"language":"Chapel","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":2,"default_branch":"master"}},"base":{"label":"chapel-lang:master","ref":"master","sha":"4e4ed8d5ff09fd608932d9cf148b2098da8b8b60","user":{"login":"chapel-lang","id":7597261,"avatar_url":"https://avatars.githubusercontent.com/u/7597261?v=3","gravatar_id":"","url":"https://api.github.com/users/chapel-lang","html_url":"https://github.com/chapel-lang","followers_url":"https://api.github.com/users/chapel-lang/followers","following_url":"https://api.github.com/users/chapel-lang/following{/other_user}","gists_url":"https://api.github.com/users/chapel-lang/gists{/gist_id}","starred_url":"https://api.github.com/users/chapel-lang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chapel-lang/subscriptions","organizations_url":"https://api.github.com/users/chapel-lang/orgs","repos_url":"https://api.github.com/users/chapel-lang/repos","events_url":"https://api.github.com/users/chapel-lang/events{/privacy}","received_events_url":"https://api.github.com/users/chapel-lang/received_events","type":"Organization","site_admin":false},"repo":{"id":21663285,"name":"chapel","full_name":"chapel-lang/chapel","owner":{"login":"chapel-lang","id":7597261,"avatar_url":"https://avatars.githubusercontent.com/u/7597261?v=3","gravatar_id":"","url":"https://api.github.com/users/chapel-lang","html_url":"https://github.com/chapel-lang","followers_url":"https://api.github.com/users/chapel-lang/followers","following_url":"https://api.github.com/users/chapel-lang/following{/other_user}","gists_url":"https://api.github.com/users/chapel-lang/gists{/gist_id}","starred_url":"https://api.github.com/users/chapel-lang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chapel-lang/subscriptions","organizations_url":"https://api.github.com/users/chapel-lang/orgs","repos_url":"https://api.github.com/users/chapel-lang/repos","events_url":"https://api.github.com/users/chapel-lang/events{/privacy}","received_events_url":"https://api.github.com/users/chapel-lang/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/chapel-lang/chapel","description":"a Productive Parallel Programming Language","fork":false,"url":"https://api.github.com/repos/chapel-lang/chapel","forks_url":"https://api.github.com/repos/chapel-lang/chapel/forks","keys_url":"https://api.github.com/repos/chapel-lang/chapel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/chapel-lang/chapel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/chapel-lang/chapel/teams","hooks_url":"https://api.github.com/repos/chapel-lang/chapel/hooks","issue_events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/events{/number}","events_url":"https://api.github.com/repos/chapel-lang/chapel/events","assignees_url":"https://api.github.com/repos/chapel-lang/chapel/assignees{/user}","branches_url":"https://api.github.com/repos/chapel-lang/chapel/branches{/branch}","tags_url":"https://api.github.com/repos/chapel-lang/chapel/tags","blobs_url":"https://api.github.com/repos/chapel-lang/chapel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/chapel-lang/chapel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/chapel-lang/chapel/git/refs{/sha}","trees_url":"https://api.github.com/repos/chapel-lang/chapel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/chapel-lang/chapel/statuses/{sha}","languages_url":"https://api.github.com/repos/chapel-lang/chapel/languages","stargazers_url":"https://api.github.com/repos/chapel-lang/chapel/stargazers","contributors_url":"https://api.github.com/repos/chapel-lang/chapel/contributors","subscribers_url":"https://api.github.com/repos/chapel-lang/chapel/subscribers","subscription_url":"https://api.github.com/repos/chapel-lang/chapel/subscription","commits_url":"https://api.github.com/repos/chapel-lang/chapel/commits{/sha}","git_commits_url":"https://api.github.com/repos/chapel-lang/chapel/git/commits{/sha}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/comments{/number}","issue_comment_url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments{/number}","contents_url":"https://api.github.com/repos/chapel-lang/chapel/contents/{+path}","compare_url":"https://api.github.com/repos/chapel-lang/chapel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/chapel-lang/chapel/merges","archive_url":"https://api.github.com/repos/chapel-lang/chapel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/chapel-lang/chapel/downloads","issues_url":"https://api.github.com/repos/chapel-lang/chapel/issues{/number}","pulls_url":"https://api.github.com/repos/chapel-lang/chapel/pulls{/number}","milestones_url":"https://api.github.com/repos/chapel-lang/chapel/milestones{/number}","notifications_url":"https://api.github.com/repos/chapel-lang/chapel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/chapel-lang/chapel/labels{/name}","releases_url":"https://api.github.com/repos/chapel-lang/chapel/releases{/id}","deployments_url":"https://api.github.com/repos/chapel-lang/chapel/deployments","created_at":"2014-07-09T18:15:54Z","updated_at":"2016-03-02T15:59:43Z","pushed_at":"2016-03-03T23:31:01Z","git_url":"git://github.com/chapel-lang/chapel.git","ssh_url":"git@github.com:chapel-lang/chapel.git","clone_url":"https://github.com/chapel-lang/chapel.git","svn_url":"https://github.com/chapel-lang/chapel","homepage":"http://chapel.cray.com/","size":441394,"stargazers_count":208,"watchers_count":208,"language":"Chapel","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":76,"mirror_url":null,"open_issues_count":11,"forks":76,"open_issues":11,"watchers":208,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380"},"html":{"href":"https://github.com/chapel-lang/chapel/pull/3380"},"issue":{"href":"https://api.github.com/repos/chapel-lang/chapel/issues/3380"},"comments":{"href":"https://api.github.com/repos/chapel-lang/chapel/issues/3380/comments"},"review_comments":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380/comments"},"review_comment":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/chapel-lang/chapel/pulls/3380/commits"},"statuses":{"href":"https://api.github.com/repos/chapel-lang/chapel/statuses/353e252fa977a761b67ad78983091f5fb7bd9d0b"}}}}