{"action":"opened","number":95,"pull_request":{"url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95","id":62164949,"html_url":"https://github.com/swcarpentry/r-novice-gapminder/pull/95","diff_url":"https://github.com/swcarpentry/r-novice-gapminder/pull/95.diff","patch_url":"https://github.com/swcarpentry/r-novice-gapminder/pull/95.patch","issue_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/95","number":95,"state":"open","locked":false,"title":"Remove square-bracket subsetting from data lessons","user":{"login":"ateucher","id":2816635,"avatar_url":"https://avatars.githubusercontent.com/u/2816635?v=3","gravatar_id":"","url":"https://api.github.com/users/ateucher","html_url":"https://github.com/ateucher","followers_url":"https://api.github.com/users/ateucher/followers","following_url":"https://api.github.com/users/ateucher/following{/other_user}","gists_url":"https://api.github.com/users/ateucher/gists{/gist_id}","starred_url":"https://api.github.com/users/ateucher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ateucher/subscriptions","organizations_url":"https://api.github.com/users/ateucher/orgs","repos_url":"https://api.github.com/users/ateucher/repos","events_url":"https://api.github.com/users/ateucher/events{/privacy}","received_events_url":"https://api.github.com/users/ateucher/received_events","type":"User","site_admin":false},"body":"As per the discussions in #89 , here is a pull request containing changes that remove square-bracket subsetting from the data lessons (04 and 05).\r\n\r\nIn addition, I:\r\n- replaced `read.table` with `read.csv`, \r\n- added a bit of an introduction to classes (very basic), \r\n- made `cats$likes_string` with 1s and 0s instead of `TRUE` and `FALSE` to help illustrate coercion.\r\n\r\nThings I didn't do:\r\n- edit the introduction lesson to introduce function usage\r\n- get rid of `rbind` and `cbind`\r\n- rework subsetting lesson. I think it still works as is, but it might be worth having another look at it, and potentially incorporate moar cats.\r\n\r\nI'm happy to work on those things, but I don't think it will happen in the very near future. If this PR looks okay, I think the overall R lesson will still work (though it would probably be worth a closer look) ... then I (or somebody else) can do those things if they are deemed necessary.","created_at":"2016-03-09T01:11:24Z","updated_at":"2016-03-09T01:11:24Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95/commits","review_comments_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95/comments","review_comment_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/comments{/number}","comments_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/95/comments","statuses_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/statuses/d313a45417f7c2679406393b010885ab21c68cc6","head":{"label":"ateucher:rm-subset-from-data","ref":"rm-subset-from-data","sha":"d313a45417f7c2679406393b010885ab21c68cc6","user":{"login":"ateucher","id":2816635,"avatar_url":"https://avatars.githubusercontent.com/u/2816635?v=3","gravatar_id":"","url":"https://api.github.com/users/ateucher","html_url":"https://github.com/ateucher","followers_url":"https://api.github.com/users/ateucher/followers","following_url":"https://api.github.com/users/ateucher/following{/other_user}","gists_url":"https://api.github.com/users/ateucher/gists{/gist_id}","starred_url":"https://api.github.com/users/ateucher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ateucher/subscriptions","organizations_url":"https://api.github.com/users/ateucher/orgs","repos_url":"https://api.github.com/users/ateucher/repos","events_url":"https://api.github.com/users/ateucher/events{/privacy}","received_events_url":"https://api.github.com/users/ateucher/received_events","type":"User","site_admin":false},"repo":{"id":51100773,"name":"r-novice-gapminder","full_name":"ateucher/r-novice-gapminder","owner":{"login":"ateucher","id":2816635,"avatar_url":"https://avatars.githubusercontent.com/u/2816635?v=3","gravatar_id":"","url":"https://api.github.com/users/ateucher","html_url":"https://github.com/ateucher","followers_url":"https://api.github.com/users/ateucher/followers","following_url":"https://api.github.com/users/ateucher/following{/other_user}","gists_url":"https://api.github.com/users/ateucher/gists{/gist_id}","starred_url":"https://api.github.com/users/ateucher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ateucher/subscriptions","organizations_url":"https://api.github.com/users/ateucher/orgs","repos_url":"https://api.github.com/users/ateucher/repos","events_url":"https://api.github.com/users/ateucher/events{/privacy}","received_events_url":"https://api.github.com/users/ateucher/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ateucher/r-novice-gapminder","description":"Introduction to R for non-programmers using gapminder data.","fork":true,"url":"https://api.github.com/repos/ateucher/r-novice-gapminder","forks_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/forks","keys_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/teams","hooks_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/hooks","issue_events_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/issues/events{/number}","events_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/events","assignees_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/assignees{/user}","branches_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/branches{/branch}","tags_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/tags","blobs_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/git/refs{/sha}","trees_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/statuses/{sha}","languages_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/languages","stargazers_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/stargazers","contributors_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/contributors","subscribers_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/subscribers","subscription_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/subscription","commits_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/commits{/sha}","git_commits_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/git/commits{/sha}","comments_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/comments{/number}","issue_comment_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/issues/comments{/number}","contents_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/contents/{+path}","compare_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/merges","archive_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/downloads","issues_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/issues{/number}","pulls_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/pulls{/number}","milestones_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/milestones{/number}","notifications_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/labels{/name}","releases_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/releases{/id}","deployments_url":"https://api.github.com/repos/ateucher/r-novice-gapminder/deployments","created_at":"2016-02-04T19:45:44Z","updated_at":"2016-02-04T19:45:45Z","pushed_at":"2016-03-09T00:53:34Z","git_url":"git://github.com/ateucher/r-novice-gapminder.git","ssh_url":"git@github.com:ateucher/r-novice-gapminder.git","clone_url":"https://github.com/ateucher/r-novice-gapminder.git","svn_url":"https://github.com/ateucher/r-novice-gapminder","homepage":null,"size":12253,"stargazers_count":0,"watchers_count":0,"language":"HTML","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"gh-pages"}},"base":{"label":"swcarpentry:gh-pages","ref":"gh-pages","sha":"70fcbd923be0dd42cd2f8bb68c2ccb639059fa2c","user":{"login":"swcarpentry","id":2238358,"avatar_url":"https://avatars.githubusercontent.com/u/2238358?v=3","gravatar_id":"","url":"https://api.github.com/users/swcarpentry","html_url":"https://github.com/swcarpentry","followers_url":"https://api.github.com/users/swcarpentry/followers","following_url":"https://api.github.com/users/swcarpentry/following{/other_user}","gists_url":"https://api.github.com/users/swcarpentry/gists{/gist_id}","starred_url":"https://api.github.com/users/swcarpentry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swcarpentry/subscriptions","organizations_url":"https://api.github.com/users/swcarpentry/orgs","repos_url":"https://api.github.com/users/swcarpentry/repos","events_url":"https://api.github.com/users/swcarpentry/events{/privacy}","received_events_url":"https://api.github.com/users/swcarpentry/received_events","type":"Organization","site_admin":false},"repo":{"id":34186021,"name":"r-novice-gapminder","full_name":"swcarpentry/r-novice-gapminder","owner":{"login":"swcarpentry","id":2238358,"avatar_url":"https://avatars.githubusercontent.com/u/2238358?v=3","gravatar_id":"","url":"https://api.github.com/users/swcarpentry","html_url":"https://github.com/swcarpentry","followers_url":"https://api.github.com/users/swcarpentry/followers","following_url":"https://api.github.com/users/swcarpentry/following{/other_user}","gists_url":"https://api.github.com/users/swcarpentry/gists{/gist_id}","starred_url":"https://api.github.com/users/swcarpentry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swcarpentry/subscriptions","organizations_url":"https://api.github.com/users/swcarpentry/orgs","repos_url":"https://api.github.com/users/swcarpentry/repos","events_url":"https://api.github.com/users/swcarpentry/events{/privacy}","received_events_url":"https://api.github.com/users/swcarpentry/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/swcarpentry/r-novice-gapminder","description":"Introduction to R for non-programmers using gapminder data.","fork":false,"url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder","forks_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/forks","keys_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/keys{/key_id}","collaborators_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/teams","hooks_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/hooks","issue_events_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/events{/number}","events_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/events","assignees_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/assignees{/user}","branches_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/branches{/branch}","tags_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/tags","blobs_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/git/refs{/sha}","trees_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/git/trees{/sha}","statuses_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/statuses/{sha}","languages_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/languages","stargazers_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/stargazers","contributors_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/contributors","subscribers_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/subscribers","subscription_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/subscription","commits_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/commits{/sha}","git_commits_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/git/commits{/sha}","comments_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/comments{/number}","issue_comment_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/comments{/number}","contents_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/contents/{+path}","compare_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/compare/{base}...{head}","merges_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/merges","archive_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/downloads","issues_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues{/number}","pulls_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls{/number}","milestones_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/milestones{/number}","notifications_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/labels{/name}","releases_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/releases{/id}","deployments_url":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/deployments","created_at":"2015-04-18T22:58:07Z","updated_at":"2016-02-26T02:33:16Z","pushed_at":"2016-02-24T19:59:10Z","git_url":"git://github.com/swcarpentry/r-novice-gapminder.git","ssh_url":"git@github.com:swcarpentry/r-novice-gapminder.git","clone_url":"https://github.com/swcarpentry/r-novice-gapminder.git","svn_url":"https://github.com/swcarpentry/r-novice-gapminder","homepage":null,"size":16609,"stargazers_count":18,"watchers_count":18,"language":"HTML","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":63,"mirror_url":null,"open_issues_count":24,"forks":63,"open_issues":24,"watchers":18,"default_branch":"gh-pages"}},"_links":{"self":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95"},"html":{"href":"https://github.com/swcarpentry/r-novice-gapminder/pull/95"},"issue":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/95"},"comments":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/issues/95/comments"},"review_comments":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95/comments"},"review_comment":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/pulls/95/commits"},"statuses":{"href":"https://api.github.com/repos/swcarpentry/r-novice-gapminder/statuses/d313a45417f7c2679406393b010885ab21c68cc6"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":30,"additions":118,"deletions":78,"changed_files":2}}