{"action":"opened","number":611,"pull_request":{"url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611","id":231837674,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMxODM3Njc0","html_url":"https://github.com/sourceryinstitute/OpenCoarrays/pull/611","diff_url":"https://github.com/sourceryinstitute/OpenCoarrays/pull/611.diff","patch_url":"https://github.com/sourceryinstitute/OpenCoarrays/pull/611.patch","issue_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/611","number":611,"state":"open","locked":false,"title":"Fix CHANGE TEAM and END TEAM deadlocks","user":{"login":"nathanweeks","id":1800812,"node_id":"MDQ6VXNlcjE4MDA4MTI=","avatar_url":"https://avatars1.githubusercontent.com/u/1800812?v=4","gravatar_id":"","url":"https://api.github.com/users/nathanweeks","html_url":"https://github.com/nathanweeks","followers_url":"https://api.github.com/users/nathanweeks/followers","following_url":"https://api.github.com/users/nathanweeks/following{/other_user}","gists_url":"https://api.github.com/users/nathanweeks/gists{/gist_id}","starred_url":"https://api.github.com/users/nathanweeks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nathanweeks/subscriptions","organizations_url":"https://api.github.com/users/nathanweeks/orgs","repos_url":"https://api.github.com/users/nathanweeks/repos","events_url":"https://api.github.com/users/nathanweeks/events{/privacy}","received_events_url":"https://api.github.com/users/nathanweeks/received_events","type":"User","site_admin":false},"body":"Fixes #610 \r\n\r\n## Summary of changes ##\r\n\r\nFor CHANGE TEAM, call MPI_Barrier() on the communicator for the images in the new team instead of the communicator corresponding to all images in its parent team. \r\n\r\nFor END TEAM, remove the extra call to MPI_Barrier() for the communicator corresponding to the parent team (keeping the barrier for the communicator for the current team).\r\n\r\n## Rationale for changes ##\r\n\r\nRemove deadlocks that could otherwise occur when CHANGE TEAM / END TEAM are called by a subset of images in the current team.\r\n\r\n## Additional info and certifications ##\r\n\r\nFortran 2018 draft (N2146):\r\n\r\n> **NOTE 11.6**\r\n> Execution of a CHANGE TEAM statement includes a synchronization of the executing image with the other images that will be in the same team after execution of the CHANGE TEAM statement. Synchronization of these images occurs again when the corresponding END TEAM statement is executed.\r\n>\r\n> If it is desired to synchronize all of the images in the team that was current when the CHANGE TEAM\r\nstatement was executed, a SYNC TEAM statement that specifies the parent team can be executed immediately after the CHANGE TEAM statement. If similar semantics are desired following the END TEAM\r\nstatement, a SYNC ALL statement could immediately follow the END TEAM statement.\r\n\r\nThis pull request (PR) is a:\r\n\r\n- [ ] Bug fix\r\n- [ ] Feature addition\r\n- [ ] Other, Please describe:\r\n\r\n### I certify that ###\r\n\r\n- [ ] I reviewed and followed the [contributing guidelines], including\r\n      - Increasing test coverage for all feature-addition PRs\r\n      - Increasing test coverage for all bug-fix PRs for which there\r\n        does not already exist a related test that failed before the PR\r\n      - At least maintaining test coverage for all other PRs\r\n      - Ensuring that all tests pass when run locally \r\n      - Naming PR  to indicate work in progress (WIP) and to attach the PR\r\n        to the appropriate bug report or feature request [issue]\r\n      - White space (no trailing white space or white space errors may\r\n        be introduced)\r\n      - Commenting code where it is non-obvious and non-trivial\r\n      - Logically atomic, self consistent and coherent commits\r\n      - Commit message content\r\n      - Waiting 24 hours before self-approving the PR to give another\r\n        OpenCoarrays developer a chance to review my proposed code\r\n","created_at":"2018-11-19T02:36:08Z","updated_at":"2018-11-19T02:36:08Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611/commits","review_comments_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611/comments","review_comment_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/comments{/number}","comments_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/611/comments","statuses_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/statuses/0368d69ea8d8e868884d62ce2ca3f8bcc29d430f","head":{"label":"nathanweeks:issue-610-fix-change-team","ref":"issue-610-fix-change-team","sha":"0368d69ea8d8e868884d62ce2ca3f8bcc29d430f","user":{"login":"nathanweeks","id":1800812,"node_id":"MDQ6VXNlcjE4MDA4MTI=","avatar_url":"https://avatars1.githubusercontent.com/u/1800812?v=4","gravatar_id":"","url":"https://api.github.com/users/nathanweeks","html_url":"https://github.com/nathanweeks","followers_url":"https://api.github.com/users/nathanweeks/followers","following_url":"https://api.github.com/users/nathanweeks/following{/other_user}","gists_url":"https://api.github.com/users/nathanweeks/gists{/gist_id}","starred_url":"https://api.github.com/users/nathanweeks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nathanweeks/subscriptions","organizations_url":"https://api.github.com/users/nathanweeks/orgs","repos_url":"https://api.github.com/users/nathanweeks/repos","events_url":"https://api.github.com/users/nathanweeks/events{/privacy}","received_events_url":"https://api.github.com/users/nathanweeks/received_events","type":"User","site_admin":false},"repo":{"id":158105170,"node_id":"MDEwOlJlcG9zaXRvcnkxNTgxMDUxNzA=","name":"OpenCoarrays","full_name":"nathanweeks/OpenCoarrays","private":false,"owner":{"login":"nathanweeks","id":1800812,"node_id":"MDQ6VXNlcjE4MDA4MTI=","avatar_url":"https://avatars1.githubusercontent.com/u/1800812?v=4","gravatar_id":"","url":"https://api.github.com/users/nathanweeks","html_url":"https://github.com/nathanweeks","followers_url":"https://api.github.com/users/nathanweeks/followers","following_url":"https://api.github.com/users/nathanweeks/following{/other_user}","gists_url":"https://api.github.com/users/nathanweeks/gists{/gist_id}","starred_url":"https://api.github.com/users/nathanweeks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nathanweeks/subscriptions","organizations_url":"https://api.github.com/users/nathanweeks/orgs","repos_url":"https://api.github.com/users/nathanweeks/repos","events_url":"https://api.github.com/users/nathanweeks/events{/privacy}","received_events_url":"https://api.github.com/users/nathanweeks/received_events","type":"User","site_admin":false},"html_url":"https://github.com/nathanweeks/OpenCoarrays","description":"A parallel application binary interface for Fortran 2018 compilers. ","fork":true,"url":"https://api.github.com/repos/nathanweeks/OpenCoarrays","forks_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/forks","keys_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/keys{/key_id}","collaborators_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/teams","hooks_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/hooks","issue_events_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/issues/events{/number}","events_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/events","assignees_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/assignees{/user}","branches_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/branches{/branch}","tags_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/tags","blobs_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/git/refs{/sha}","trees_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/git/trees{/sha}","statuses_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/statuses/{sha}","languages_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/languages","stargazers_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/stargazers","contributors_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/contributors","subscribers_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/subscribers","subscription_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/subscription","commits_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/commits{/sha}","git_commits_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/git/commits{/sha}","comments_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/comments{/number}","issue_comment_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/issues/comments{/number}","contents_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/contents/{+path}","compare_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/compare/{base}...{head}","merges_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/merges","archive_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/downloads","issues_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/issues{/number}","pulls_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/pulls{/number}","milestones_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/milestones{/number}","notifications_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/labels{/name}","releases_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/releases{/id}","deployments_url":"https://api.github.com/repos/nathanweeks/OpenCoarrays/deployments","created_at":"2018-11-18T16:45:22Z","updated_at":"2018-11-18T16:45:25Z","pushed_at":"2018-11-19T02:01:03Z","git_url":"git://github.com/nathanweeks/OpenCoarrays.git","ssh_url":"git@github.com:nathanweeks/OpenCoarrays.git","clone_url":"https://github.com/nathanweeks/OpenCoarrays.git","svn_url":"https://github.com/nathanweeks/OpenCoarrays","homepage":"http://www.opencoarrays.org","size":8022,"stargazers_count":0,"watchers_count":0,"language":"Fortran","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":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"sourceryinstitute:master","ref":"master","sha":"003efcbe5b5e6a6246a07112dbc5c5b43e563af6","user":{"login":"sourceryinstitute","id":1010472,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEwMTA0NzI=","avatar_url":"https://avatars3.githubusercontent.com/u/1010472?v=4","gravatar_id":"","url":"https://api.github.com/users/sourceryinstitute","html_url":"https://github.com/sourceryinstitute","followers_url":"https://api.github.com/users/sourceryinstitute/followers","following_url":"https://api.github.com/users/sourceryinstitute/following{/other_user}","gists_url":"https://api.github.com/users/sourceryinstitute/gists{/gist_id}","starred_url":"https://api.github.com/users/sourceryinstitute/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sourceryinstitute/subscriptions","organizations_url":"https://api.github.com/users/sourceryinstitute/orgs","repos_url":"https://api.github.com/users/sourceryinstitute/repos","events_url":"https://api.github.com/users/sourceryinstitute/events{/privacy}","received_events_url":"https://api.github.com/users/sourceryinstitute/received_events","type":"Organization","site_admin":false},"repo":{"id":34347971,"node_id":"MDEwOlJlcG9zaXRvcnkzNDM0Nzk3MQ==","name":"OpenCoarrays","full_name":"sourceryinstitute/OpenCoarrays","private":false,"owner":{"login":"sourceryinstitute","id":1010472,"node_id":"MDEyOk9yZ2FuaXphdGlvbjEwMTA0NzI=","avatar_url":"https://avatars3.githubusercontent.com/u/1010472?v=4","gravatar_id":"","url":"https://api.github.com/users/sourceryinstitute","html_url":"https://github.com/sourceryinstitute","followers_url":"https://api.github.com/users/sourceryinstitute/followers","following_url":"https://api.github.com/users/sourceryinstitute/following{/other_user}","gists_url":"https://api.github.com/users/sourceryinstitute/gists{/gist_id}","starred_url":"https://api.github.com/users/sourceryinstitute/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sourceryinstitute/subscriptions","organizations_url":"https://api.github.com/users/sourceryinstitute/orgs","repos_url":"https://api.github.com/users/sourceryinstitute/repos","events_url":"https://api.github.com/users/sourceryinstitute/events{/privacy}","received_events_url":"https://api.github.com/users/sourceryinstitute/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/sourceryinstitute/OpenCoarrays","description":"A parallel application binary interface for Fortran 2018 compilers. ","fork":false,"url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays","forks_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/forks","keys_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/teams","hooks_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/hooks","issue_events_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/events{/number}","events_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/events","assignees_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/assignees{/user}","branches_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/branches{/branch}","tags_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/tags","blobs_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/git/refs{/sha}","trees_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/statuses/{sha}","languages_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/languages","stargazers_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/stargazers","contributors_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/contributors","subscribers_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/subscribers","subscription_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/subscription","commits_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/commits{/sha}","git_commits_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/git/commits{/sha}","comments_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/comments{/number}","issue_comment_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/comments{/number}","contents_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/contents/{+path}","compare_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/merges","archive_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/downloads","issues_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues{/number}","pulls_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls{/number}","milestones_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/milestones{/number}","notifications_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/labels{/name}","releases_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/releases{/id}","deployments_url":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/deployments","created_at":"2015-04-21T19:32:26Z","updated_at":"2018-11-08T21:11:37Z","pushed_at":"2018-11-08T22:21:35Z","git_url":"git://github.com/sourceryinstitute/OpenCoarrays.git","ssh_url":"git@github.com:sourceryinstitute/OpenCoarrays.git","clone_url":"https://github.com/sourceryinstitute/OpenCoarrays.git","svn_url":"https://github.com/sourceryinstitute/OpenCoarrays","homepage":"http://www.opencoarrays.org","size":8021,"stargazers_count":87,"watchers_count":87,"language":"Fortran","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":29,"mirror_url":null,"archived":false,"open_issues_count":38,"license":{"key":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":29,"open_issues":38,"watchers":87,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611"},"html":{"href":"https://github.com/sourceryinstitute/OpenCoarrays/pull/611"},"issue":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/611"},"comments":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/issues/611/comments"},"review_comments":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611/comments"},"review_comment":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/pulls/611/commits"},"statuses":{"href":"https://api.github.com/repos/sourceryinstitute/OpenCoarrays/statuses/0368d69ea8d8e868884d62ce2ca3f8bcc29d430f"}},"author_association":"NONE","merged":false,"mergeable":null,"rebaseable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"maintainer_can_modify":true,"commits":1,"additions":39,"deletions":3,"changed_files":4}}
{"actor":{"display_login":"nathanweeks"}}