{"action":"created","comment":{"url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/comments/88570145","pull_request_review_id":9141330,"id":88570145,"diff_hunk":"@@ -24,14 +24,14 @@\n         \"SAM\" : {'domain':cdutil.region.domain(latitude=(-20.,-90,'ccb'),longitude=(0,360,'ccb'))},\n         \"PNA\" : {'domain':cdutil.region.domain(latitude=(20.,85,'ccb'),longitude=(120,240,'ccb'))},\n         \"PDO\" : {'domain':cdutil.region.domain(latitude=(20.,70,'ccb'),longitude=(110,260,'ccb'))},\n-# Below is for monsoon regions \n-          'AllMn':{'domain':cdutil.region.domain(latitude=(-45.,45,'ccb'),longitude=(0,360,'ccb'))}, # All monsoon domains\n-          'NAMn' :{'domain':cdutil.region.domain(latitude=(0.,45,'ccb'),longitude=(210,310,'ccb'))}, # North American Monsoon\n-          'SAMn' :{'domain':cdutil.region.domain(latitude=(-45.,0,'ccb'),longitude=(240,330,'ccb'))}, # South American Monsoon\n-          'NAFn':{'domain':cdutil.region.domain(latitude=(0.,45,'ccb'),longitude=(310,60,'ccb'))},  # North African Monsoon\n-          'SAFn':{'domain':cdutil.region.domain(latitude=(-45.,0.,'ccb'),longitude=(0.,90,'ccb'))}, # South African Monsoon\n-          'ASMn' :{'domain':cdutil.region.domain(latitude=(0.,45.,'ccb'),longitude=(60.,180.,'ccb'))}, # Asian Summer Monsoon\n-          'AUSMn':{'domain':cdutil.region.domain(latitude=(-45.,0,'ccb'),longitude=(90.,160.,'ccb'))}, # Australian Monsoon\n+# Below is for monsoon domains\n+        'AllM':{'domain':cdutil.region.domain(latitude=(-45.,45.,'ccb'),longitude=(0.,360.,'ccb'))}, # All monsoon domains\n+        'NAMM' :{'domain':cdutil.region.domain(latitude=(0.,45.,'ccb'),longitude=(210.,310.,'ccb'))}, # North American Monsoon\n+        'SAMM' :{'domain':cdutil.region.domain(latitude=(-45.,0.,'ccb'),longitude=(240.,330.,'ccb'))}, # South American Monsoon\n+        'NAFM':{'domain':cdutil.region.domain(latitude=(0.,45.,'ccb'),longitude=(310.,60.,'ccb'))},  # North African Monsoon\n+        'SAFM':{'domain':cdutil.region.domain(latitude=(-45.,0.,'ccb'),longitude=(0.,90.,'ccb'))}, # South African Monsoon\n+        'ASM' :{'domain':cdutil.region.domain(latitude=(0.,45.,'ccb'),longitude=(60.,180.,'ccb'))}, # Asian Summer Monsoon","path":"share/default_regions.py","position":null,"original_position":18,"commit_id":"1ed03bf5d7b1276ddf09b1eaa4fc0bcc57a81af4","original_commit_id":"3ad81af4492c939ce429e2e1533c6f516ffb8a82","user":{"login":"durack1","id":3229632,"avatar_url":"https://avatars.githubusercontent.com/u/3229632?v=3","gravatar_id":"","url":"https://api.github.com/users/durack1","html_url":"https://github.com/durack1","followers_url":"https://api.github.com/users/durack1/followers","following_url":"https://api.github.com/users/durack1/following{/other_user}","gists_url":"https://api.github.com/users/durack1/gists{/gist_id}","starred_url":"https://api.github.com/users/durack1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/durack1/subscriptions","organizations_url":"https://api.github.com/users/durack1/orgs","repos_url":"https://api.github.com/users/durack1/repos","events_url":"https://api.github.com/users/durack1/events{/privacy}","received_events_url":"https://api.github.com/users/durack1/received_events","type":"User","site_admin":false},"body":"This seems inconsistent - there is a space in some and not others.. Code will work, just doesn't look that clean","created_at":"2016-11-17T23:22:55Z","updated_at":"2016-11-17T23:22:55Z","html_url":"https://github.com/PCMDI/pcmdi_metrics/pull/433#discussion_r88570145","pull_request_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433","_links":{"self":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/comments/88570145"},"html":{"href":"https://github.com/PCMDI/pcmdi_metrics/pull/433#discussion_r88570145"},"pull_request":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433"}}},"pull_request":{"url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433","id":89515561,"html_url":"https://github.com/PCMDI/pcmdi_metrics/pull/433","diff_url":"https://github.com/PCMDI/pcmdi_metrics/pull/433.diff","patch_url":"https://github.com/PCMDI/pcmdi_metrics/pull/433.patch","issue_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/433","number":433,"state":"open","locked":false,"title":"added monsoon regions to reg domains in share ","user":{"login":"gleckler1","id":4553389,"avatar_url":"https://avatars.githubusercontent.com/u/4553389?v=3","gravatar_id":"","url":"https://api.github.com/users/gleckler1","html_url":"https://github.com/gleckler1","followers_url":"https://api.github.com/users/gleckler1/followers","following_url":"https://api.github.com/users/gleckler1/following{/other_user}","gists_url":"https://api.github.com/users/gleckler1/gists{/gist_id}","starred_url":"https://api.github.com/users/gleckler1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gleckler1/subscriptions","organizations_url":"https://api.github.com/users/gleckler1/orgs","repos_url":"https://api.github.com/users/gleckler1/repos","events_url":"https://api.github.com/users/gleckler1/events{/privacy}","received_events_url":"https://api.github.com/users/gleckler1/received_events","type":"User","site_admin":false},"body":"Only update in this branch is to the regional dictionary: https://github.com/PCMDI/pcmdi_metrics/blob/432_pjg_monsoon_dom/share/default_regions.py\n","created_at":"2016-10-16T04:29:30Z","updated_at":"2016-11-17T23:22:55Z","closed_at":null,"merged_at":null,"merge_commit_sha":"7a0674bafe9855b7577cb9b9f2c2f487256b72b2","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433/commits","review_comments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433/comments","review_comment_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/comments{/number}","comments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/433/comments","statuses_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/statuses/1ed03bf5d7b1276ddf09b1eaa4fc0bcc57a81af4","head":{"label":"PCMDI:432_pjg_monsoon_dom","ref":"432_pjg_monsoon_dom","sha":"1ed03bf5d7b1276ddf09b1eaa4fc0bcc57a81af4","user":{"login":"PCMDI","id":6098305,"avatar_url":"https://avatars.githubusercontent.com/u/6098305?v=3","gravatar_id":"","url":"https://api.github.com/users/PCMDI","html_url":"https://github.com/PCMDI","followers_url":"https://api.github.com/users/PCMDI/followers","following_url":"https://api.github.com/users/PCMDI/following{/other_user}","gists_url":"https://api.github.com/users/PCMDI/gists{/gist_id}","starred_url":"https://api.github.com/users/PCMDI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PCMDI/subscriptions","organizations_url":"https://api.github.com/users/PCMDI/orgs","repos_url":"https://api.github.com/users/PCMDI/repos","events_url":"https://api.github.com/users/PCMDI/events{/privacy}","received_events_url":"https://api.github.com/users/PCMDI/received_events","type":"Organization","site_admin":false},"repo":{"id":10345204,"name":"pcmdi_metrics","full_name":"PCMDI/pcmdi_metrics","owner":{"login":"PCMDI","id":6098305,"avatar_url":"https://avatars.githubusercontent.com/u/6098305?v=3","gravatar_id":"","url":"https://api.github.com/users/PCMDI","html_url":"https://github.com/PCMDI","followers_url":"https://api.github.com/users/PCMDI/followers","following_url":"https://api.github.com/users/PCMDI/following{/other_user}","gists_url":"https://api.github.com/users/PCMDI/gists{/gist_id}","starred_url":"https://api.github.com/users/PCMDI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PCMDI/subscriptions","organizations_url":"https://api.github.com/users/PCMDI/orgs","repos_url":"https://api.github.com/users/PCMDI/repos","events_url":"https://api.github.com/users/PCMDI/events{/privacy}","received_events_url":"https://api.github.com/users/PCMDI/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/PCMDI/pcmdi_metrics","description":"Self contained packages to run PCMDI Metrics","fork":false,"url":"https://api.github.com/repos/PCMDI/pcmdi_metrics","forks_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/forks","keys_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/keys{/key_id}","collaborators_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/teams","hooks_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/hooks","issue_events_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/events{/number}","events_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/events","assignees_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/assignees{/user}","branches_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/branches{/branch}","tags_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/tags","blobs_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/refs{/sha}","trees_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/trees{/sha}","statuses_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/statuses/{sha}","languages_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/languages","stargazers_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/stargazers","contributors_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/contributors","subscribers_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/subscribers","subscription_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/subscription","commits_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/commits{/sha}","git_commits_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/commits{/sha}","comments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/comments{/number}","issue_comment_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/comments{/number}","contents_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/contents/{+path}","compare_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/compare/{base}...{head}","merges_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/merges","archive_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/downloads","issues_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues{/number}","pulls_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls{/number}","milestones_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/milestones{/number}","notifications_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/labels{/name}","releases_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/releases{/id}","deployments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/deployments","created_at":"2013-05-28T20:48:38Z","updated_at":"2016-10-24T18:51:25Z","pushed_at":"2016-11-17T23:21:57Z","git_url":"git://github.com/PCMDI/pcmdi_metrics.git","ssh_url":"git@github.com:PCMDI/pcmdi_metrics.git","clone_url":"https://github.com/PCMDI/pcmdi_metrics.git","svn_url":"https://github.com/PCMDI/pcmdi_metrics","homepage":"","size":21797,"stargazers_count":16,"watchers_count":16,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":10,"mirror_url":null,"open_issues_count":47,"forks":10,"open_issues":47,"watchers":16,"default_branch":"master"}},"base":{"label":"PCMDI:master","ref":"master","sha":"5889cd8d9702d1c0105c88d8e8bb52a039348508","user":{"login":"PCMDI","id":6098305,"avatar_url":"https://avatars.githubusercontent.com/u/6098305?v=3","gravatar_id":"","url":"https://api.github.com/users/PCMDI","html_url":"https://github.com/PCMDI","followers_url":"https://api.github.com/users/PCMDI/followers","following_url":"https://api.github.com/users/PCMDI/following{/other_user}","gists_url":"https://api.github.com/users/PCMDI/gists{/gist_id}","starred_url":"https://api.github.com/users/PCMDI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PCMDI/subscriptions","organizations_url":"https://api.github.com/users/PCMDI/orgs","repos_url":"https://api.github.com/users/PCMDI/repos","events_url":"https://api.github.com/users/PCMDI/events{/privacy}","received_events_url":"https://api.github.com/users/PCMDI/received_events","type":"Organization","site_admin":false},"repo":{"id":10345204,"name":"pcmdi_metrics","full_name":"PCMDI/pcmdi_metrics","owner":{"login":"PCMDI","id":6098305,"avatar_url":"https://avatars.githubusercontent.com/u/6098305?v=3","gravatar_id":"","url":"https://api.github.com/users/PCMDI","html_url":"https://github.com/PCMDI","followers_url":"https://api.github.com/users/PCMDI/followers","following_url":"https://api.github.com/users/PCMDI/following{/other_user}","gists_url":"https://api.github.com/users/PCMDI/gists{/gist_id}","starred_url":"https://api.github.com/users/PCMDI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PCMDI/subscriptions","organizations_url":"https://api.github.com/users/PCMDI/orgs","repos_url":"https://api.github.com/users/PCMDI/repos","events_url":"https://api.github.com/users/PCMDI/events{/privacy}","received_events_url":"https://api.github.com/users/PCMDI/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/PCMDI/pcmdi_metrics","description":"Self contained packages to run PCMDI Metrics","fork":false,"url":"https://api.github.com/repos/PCMDI/pcmdi_metrics","forks_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/forks","keys_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/keys{/key_id}","collaborators_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/teams","hooks_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/hooks","issue_events_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/events{/number}","events_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/events","assignees_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/assignees{/user}","branches_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/branches{/branch}","tags_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/tags","blobs_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/refs{/sha}","trees_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/trees{/sha}","statuses_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/statuses/{sha}","languages_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/languages","stargazers_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/stargazers","contributors_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/contributors","subscribers_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/subscribers","subscription_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/subscription","commits_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/commits{/sha}","git_commits_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/git/commits{/sha}","comments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/comments{/number}","issue_comment_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/comments{/number}","contents_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/contents/{+path}","compare_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/compare/{base}...{head}","merges_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/merges","archive_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/downloads","issues_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues{/number}","pulls_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls{/number}","milestones_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/milestones{/number}","notifications_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/labels{/name}","releases_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/releases{/id}","deployments_url":"https://api.github.com/repos/PCMDI/pcmdi_metrics/deployments","created_at":"2013-05-28T20:48:38Z","updated_at":"2016-10-24T18:51:25Z","pushed_at":"2016-11-17T23:21:57Z","git_url":"git://github.com/PCMDI/pcmdi_metrics.git","ssh_url":"git@github.com:PCMDI/pcmdi_metrics.git","clone_url":"https://github.com/PCMDI/pcmdi_metrics.git","svn_url":"https://github.com/PCMDI/pcmdi_metrics","homepage":"","size":21797,"stargazers_count":16,"watchers_count":16,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":10,"mirror_url":null,"open_issues_count":47,"forks":10,"open_issues":47,"watchers":16,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433"},"html":{"href":"https://github.com/PCMDI/pcmdi_metrics/pull/433"},"issue":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/433"},"comments":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/issues/433/comments"},"review_comments":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433/comments"},"review_comment":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/pulls/433/commits"},"statuses":{"href":"https://api.github.com/repos/PCMDI/pcmdi_metrics/statuses/1ed03bf5d7b1276ddf09b1eaa4fc0bcc57a81af4"}}}}
{"actor":{"display_login":"durack1"}}