{"action":"created","comment":{"url":"https://api.github.com/repos/Unidata/MetPy/pulls/comments/143207852","pull_request_review_id":67689835,"id":143207852,"diff_hunk":"@@ -85,25 +69,29 @@\n \n # Map weather strings to WMO codes, which we can use to convert to symbols\n # Only use the first symbol if there are multiple\n-wx_text = [s.decode('ascii') for s in data['weather']]\n wx_codes = {'': 0, 'HZ': 5, 'BR': 10, '-DZ': 51, 'DZ': 53, '+DZ': 55,\n             '-RA': 61, 'RA': 63, '+RA': 65, '-SN': 71, 'SN': 73, '+SN': 75}\n-wx = [wx_codes[s.split()[0] if ' ' in s else s] for s in wx_text]\n+wx = [wx_codes[s.split()[0] if ' ' in s else s] for s in data['weather'].fillna('')]","path":"examples/plots/Station_Plot.py","position":91,"original_position":91,"commit_id":"e978ceff8aca730264f0cea09852e503b2829820","original_commit_id":"e978ceff8aca730264f0cea09852e503b2829820","user":{"login":"jrleeman","id":325269,"avatar_url":"https://avatars3.githubusercontent.com/u/325269?v=4","gravatar_id":"","url":"https://api.github.com/users/jrleeman","html_url":"https://github.com/jrleeman","followers_url":"https://api.github.com/users/jrleeman/followers","following_url":"https://api.github.com/users/jrleeman/following{/other_user}","gists_url":"https://api.github.com/users/jrleeman/gists{/gist_id}","starred_url":"https://api.github.com/users/jrleeman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jrleeman/subscriptions","organizations_url":"https://api.github.com/users/jrleeman/orgs","repos_url":"https://api.github.com/users/jrleeman/repos","events_url":"https://api.github.com/users/jrleeman/events{/privacy}","received_events_url":"https://api.github.com/users/jrleeman/received_events","type":"User","site_admin":false},"body":"Should the `fillna` be a separate step right after the data are read in? Might make sense to have all the data format munging in one spot.","created_at":"2017-10-06T14:41:43Z","updated_at":"2017-10-06T14:42:36Z","html_url":"https://github.com/Unidata/MetPy/pull/578#discussion_r143207852","pull_request_url":"https://api.github.com/repos/Unidata/MetPy/pulls/578","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/comments/143207852"},"html":{"href":"https://github.com/Unidata/MetPy/pull/578#discussion_r143207852"},"pull_request":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/578"}}},"pull_request":{"url":"https://api.github.com/repos/Unidata/MetPy/pulls/578","id":145009715,"html_url":"https://github.com/Unidata/MetPy/pull/578","diff_url":"https://github.com/Unidata/MetPy/pull/578.diff","patch_url":"https://github.com/Unidata/MetPy/pull/578.patch","issue_url":"https://api.github.com/repos/Unidata/MetPy/issues/578","number":578,"state":"open","locked":false,"title":"Fix examples","user":{"login":"dopplershift","id":221526,"avatar_url":"https://avatars2.githubusercontent.com/u/221526?v=4","gravatar_id":"","url":"https://api.github.com/users/dopplershift","html_url":"https://github.com/dopplershift","followers_url":"https://api.github.com/users/dopplershift/followers","following_url":"https://api.github.com/users/dopplershift/following{/other_user}","gists_url":"https://api.github.com/users/dopplershift/gists{/gist_id}","starred_url":"https://api.github.com/users/dopplershift/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dopplershift/subscriptions","organizations_url":"https://api.github.com/users/dopplershift/orgs","repos_url":"https://api.github.com/users/dopplershift/repos","events_url":"https://api.github.com/users/dopplershift/events{/privacy}","received_events_url":"https://api.github.com/users/dopplershift/received_events","type":"User","site_admin":false},"body":"Some of our examples were warning when using the station data because we weren't correctly flagging some missing values. Of course the right solution is to significantly re-write these examples to use pandas instead of `numpy.loadtxt`. I'm also taking this time to clean up some copyright years and update the list of authors.","created_at":"2017-10-05T20:12:41Z","updated_at":"2017-10-06T14:42:36Z","closed_at":null,"merged_at":null,"merge_commit_sha":"32dcba2258cdc3454788f779d7aed11c078d434a","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/Unidata/MetPy/pulls/578/commits","review_comments_url":"https://api.github.com/repos/Unidata/MetPy/pulls/578/comments","review_comment_url":"https://api.github.com/repos/Unidata/MetPy/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Unidata/MetPy/issues/578/comments","statuses_url":"https://api.github.com/repos/Unidata/MetPy/statuses/e978ceff8aca730264f0cea09852e503b2829820","head":{"label":"dopplershift:fix-example","ref":"fix-example","sha":"e978ceff8aca730264f0cea09852e503b2829820","user":{"login":"dopplershift","id":221526,"avatar_url":"https://avatars2.githubusercontent.com/u/221526?v=4","gravatar_id":"","url":"https://api.github.com/users/dopplershift","html_url":"https://github.com/dopplershift","followers_url":"https://api.github.com/users/dopplershift/followers","following_url":"https://api.github.com/users/dopplershift/following{/other_user}","gists_url":"https://api.github.com/users/dopplershift/gists{/gist_id}","starred_url":"https://api.github.com/users/dopplershift/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dopplershift/subscriptions","organizations_url":"https://api.github.com/users/dopplershift/orgs","repos_url":"https://api.github.com/users/dopplershift/repos","events_url":"https://api.github.com/users/dopplershift/events{/privacy}","received_events_url":"https://api.github.com/users/dopplershift/received_events","type":"User","site_admin":false},"repo":{"id":18279631,"name":"MetPy","full_name":"dopplershift/MetPy","owner":{"login":"dopplershift","id":221526,"avatar_url":"https://avatars2.githubusercontent.com/u/221526?v=4","gravatar_id":"","url":"https://api.github.com/users/dopplershift","html_url":"https://github.com/dopplershift","followers_url":"https://api.github.com/users/dopplershift/followers","following_url":"https://api.github.com/users/dopplershift/following{/other_user}","gists_url":"https://api.github.com/users/dopplershift/gists{/gist_id}","starred_url":"https://api.github.com/users/dopplershift/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dopplershift/subscriptions","organizations_url":"https://api.github.com/users/dopplershift/orgs","repos_url":"https://api.github.com/users/dopplershift/repos","events_url":"https://api.github.com/users/dopplershift/events{/privacy}","received_events_url":"https://api.github.com/users/dopplershift/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/dopplershift/MetPy","description":"A Python Package for Meteorological Data","fork":true,"url":"https://api.github.com/repos/dopplershift/MetPy","forks_url":"https://api.github.com/repos/dopplershift/MetPy/forks","keys_url":"https://api.github.com/repos/dopplershift/MetPy/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dopplershift/MetPy/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dopplershift/MetPy/teams","hooks_url":"https://api.github.com/repos/dopplershift/MetPy/hooks","issue_events_url":"https://api.github.com/repos/dopplershift/MetPy/issues/events{/number}","events_url":"https://api.github.com/repos/dopplershift/MetPy/events","assignees_url":"https://api.github.com/repos/dopplershift/MetPy/assignees{/user}","branches_url":"https://api.github.com/repos/dopplershift/MetPy/branches{/branch}","tags_url":"https://api.github.com/repos/dopplershift/MetPy/tags","blobs_url":"https://api.github.com/repos/dopplershift/MetPy/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dopplershift/MetPy/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dopplershift/MetPy/git/refs{/sha}","trees_url":"https://api.github.com/repos/dopplershift/MetPy/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dopplershift/MetPy/statuses/{sha}","languages_url":"https://api.github.com/repos/dopplershift/MetPy/languages","stargazers_url":"https://api.github.com/repos/dopplershift/MetPy/stargazers","contributors_url":"https://api.github.com/repos/dopplershift/MetPy/contributors","subscribers_url":"https://api.github.com/repos/dopplershift/MetPy/subscribers","subscription_url":"https://api.github.com/repos/dopplershift/MetPy/subscription","commits_url":"https://api.github.com/repos/dopplershift/MetPy/commits{/sha}","git_commits_url":"https://api.github.com/repos/dopplershift/MetPy/git/commits{/sha}","comments_url":"https://api.github.com/repos/dopplershift/MetPy/comments{/number}","issue_comment_url":"https://api.github.com/repos/dopplershift/MetPy/issues/comments{/number}","contents_url":"https://api.github.com/repos/dopplershift/MetPy/contents/{+path}","compare_url":"https://api.github.com/repos/dopplershift/MetPy/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dopplershift/MetPy/merges","archive_url":"https://api.github.com/repos/dopplershift/MetPy/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dopplershift/MetPy/downloads","issues_url":"https://api.github.com/repos/dopplershift/MetPy/issues{/number}","pulls_url":"https://api.github.com/repos/dopplershift/MetPy/pulls{/number}","milestones_url":"https://api.github.com/repos/dopplershift/MetPy/milestones{/number}","notifications_url":"https://api.github.com/repos/dopplershift/MetPy/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dopplershift/MetPy/labels{/name}","releases_url":"https://api.github.com/repos/dopplershift/MetPy/releases{/id}","deployments_url":"https://api.github.com/repos/dopplershift/MetPy/deployments","created_at":"2014-03-31T02:32:25Z","updated_at":"2016-10-31T16:57:45Z","pushed_at":"2017-10-05T20:11:52Z","git_url":"git://github.com/dopplershift/MetPy.git","ssh_url":"git@github.com:dopplershift/MetPy.git","clone_url":"https://github.com/dopplershift/MetPy.git","svn_url":"https://github.com/dopplershift/MetPy","homepage":"","size":139601,"stargazers_count":2,"watchers_count":2,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":2,"mirror_url":null,"open_issues_count":0,"forks":2,"open_issues":0,"watchers":2,"default_branch":"master"}},"base":{"label":"Unidata:master","ref":"master","sha":"270284e4af1a0575ab5147913748e5c9fcbcdf89","user":{"login":"Unidata","id":613345,"avatar_url":"https://avatars3.githubusercontent.com/u/613345?v=4","gravatar_id":"","url":"https://api.github.com/users/Unidata","html_url":"https://github.com/Unidata","followers_url":"https://api.github.com/users/Unidata/followers","following_url":"https://api.github.com/users/Unidata/following{/other_user}","gists_url":"https://api.github.com/users/Unidata/gists{/gist_id}","starred_url":"https://api.github.com/users/Unidata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Unidata/subscriptions","organizations_url":"https://api.github.com/users/Unidata/orgs","repos_url":"https://api.github.com/users/Unidata/repos","events_url":"https://api.github.com/users/Unidata/events{/privacy}","received_events_url":"https://api.github.com/users/Unidata/received_events","type":"Organization","site_admin":false},"repo":{"id":1409621,"name":"MetPy","full_name":"Unidata/MetPy","owner":{"login":"Unidata","id":613345,"avatar_url":"https://avatars3.githubusercontent.com/u/613345?v=4","gravatar_id":"","url":"https://api.github.com/users/Unidata","html_url":"https://github.com/Unidata","followers_url":"https://api.github.com/users/Unidata/followers","following_url":"https://api.github.com/users/Unidata/following{/other_user}","gists_url":"https://api.github.com/users/Unidata/gists{/gist_id}","starred_url":"https://api.github.com/users/Unidata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Unidata/subscriptions","organizations_url":"https://api.github.com/users/Unidata/orgs","repos_url":"https://api.github.com/users/Unidata/repos","events_url":"https://api.github.com/users/Unidata/events{/privacy}","received_events_url":"https://api.github.com/users/Unidata/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Unidata/MetPy","description":"MetPy is a collection of tools in Python for reading, visualizing and performing calculations with weather data.","fork":false,"url":"https://api.github.com/repos/Unidata/MetPy","forks_url":"https://api.github.com/repos/Unidata/MetPy/forks","keys_url":"https://api.github.com/repos/Unidata/MetPy/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Unidata/MetPy/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Unidata/MetPy/teams","hooks_url":"https://api.github.com/repos/Unidata/MetPy/hooks","issue_events_url":"https://api.github.com/repos/Unidata/MetPy/issues/events{/number}","events_url":"https://api.github.com/repos/Unidata/MetPy/events","assignees_url":"https://api.github.com/repos/Unidata/MetPy/assignees{/user}","branches_url":"https://api.github.com/repos/Unidata/MetPy/branches{/branch}","tags_url":"https://api.github.com/repos/Unidata/MetPy/tags","blobs_url":"https://api.github.com/repos/Unidata/MetPy/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Unidata/MetPy/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Unidata/MetPy/git/refs{/sha}","trees_url":"https://api.github.com/repos/Unidata/MetPy/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Unidata/MetPy/statuses/{sha}","languages_url":"https://api.github.com/repos/Unidata/MetPy/languages","stargazers_url":"https://api.github.com/repos/Unidata/MetPy/stargazers","contributors_url":"https://api.github.com/repos/Unidata/MetPy/contributors","subscribers_url":"https://api.github.com/repos/Unidata/MetPy/subscribers","subscription_url":"https://api.github.com/repos/Unidata/MetPy/subscription","commits_url":"https://api.github.com/repos/Unidata/MetPy/commits{/sha}","git_commits_url":"https://api.github.com/repos/Unidata/MetPy/git/commits{/sha}","comments_url":"https://api.github.com/repos/Unidata/MetPy/comments{/number}","issue_comment_url":"https://api.github.com/repos/Unidata/MetPy/issues/comments{/number}","contents_url":"https://api.github.com/repos/Unidata/MetPy/contents/{+path}","compare_url":"https://api.github.com/repos/Unidata/MetPy/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Unidata/MetPy/merges","archive_url":"https://api.github.com/repos/Unidata/MetPy/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Unidata/MetPy/downloads","issues_url":"https://api.github.com/repos/Unidata/MetPy/issues{/number}","pulls_url":"https://api.github.com/repos/Unidata/MetPy/pulls{/number}","milestones_url":"https://api.github.com/repos/Unidata/MetPy/milestones{/number}","notifications_url":"https://api.github.com/repos/Unidata/MetPy/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Unidata/MetPy/labels{/name}","releases_url":"https://api.github.com/repos/Unidata/MetPy/releases{/id}","deployments_url":"https://api.github.com/repos/Unidata/MetPy/deployments","created_at":"2011-02-25T04:20:25Z","updated_at":"2017-10-06T10:23:47Z","pushed_at":"2017-10-05T20:12:42Z","git_url":"git://github.com/Unidata/MetPy.git","ssh_url":"git@github.com:Unidata/MetPy.git","clone_url":"https://github.com/Unidata/MetPy.git","svn_url":"https://github.com/Unidata/MetPy","homepage":"https://unidata.github.io/MetPy","size":151240,"stargazers_count":202,"watchers_count":202,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":97,"mirror_url":null,"open_issues_count":114,"forks":97,"open_issues":114,"watchers":202,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/578"},"html":{"href":"https://github.com/Unidata/MetPy/pull/578"},"issue":{"href":"https://api.github.com/repos/Unidata/MetPy/issues/578"},"comments":{"href":"https://api.github.com/repos/Unidata/MetPy/issues/578/comments"},"review_comments":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/578/comments"},"review_comment":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Unidata/MetPy/pulls/578/commits"},"statuses":{"href":"https://api.github.com/repos/Unidata/MetPy/statuses/e978ceff8aca730264f0cea09852e503b2829820"}},"author_association":"OWNER"}}
{"actor":{"display_login":"jrleeman"}}