GitHub PullRequestReviewCommentEvent at 2014-11-10 17:40:38
Repository: site
Actor: lwjohnst86
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/swcarpentry/site/pulls/comments/20098145","id":20098145,"diff_hunk":"@@ -0,0 +1,130 @@\n+---\n+layout: blog\n+root: ../../..\n+author: Pauline Barmby\n+title: \"Workshop Summary: Gerstein Science Information Centre, University of Toronto\"\n+date: 2014-11-08\n+time: \"09:30:00\"\n+category: [\"Workshops\", \"University of Toronto\"]\n+---\n+<!-- start excerpt -->\n+<p>\n+  Last week we ran a Python-based SWC workshop at the University of\n+  Toronto's <a href=\"\">Gerstein Science Information Centre</a>. The advertised audience\n+was graduate students, post-docs and other researchers in science, engineering and medicine at the University of Toronto.\n+</p>\n+<!-- end excerpt -->\n+<p>\n+The workshop had about 20 learners. Most were grad students or postdocs\n+in life sciences. The pre-workshop assessment\n+said that about three-quarters of learners had some programming experience\n+with about half of those having some Python; maybe half of the total\n+had some familiarity with the command line.\n+Almost none had experience with version control or SQL. \n+The instructors were P. Barmby, G. Wilson, and T. Guignard, helpers were\n+S. Rahmani, L. Johnston, D. Neilsen, T. Wright, and X. Fei.\n+Our host was Erica Lenton.\n+</p>\n+\n+<p>\n+Nearly everyone had done the software installation prior to the workshop (!),\n+and not too many problems were reported.  The laptops in use\n+were roughly 50% Mac, 40% Windows, 10% Linux.\n+A handful of learners ran into the Mac Git problem, on which more below.\n+</p>\n+\n+<p>\n+The physical layout of the room was not quite ideal: the projector was pretty \n+close to the screen and so the projected area on the screen was quite small.\n+This meant that we had to use quite large fonts so that people at the back of the\n+room could see, and only a few lines at a time could be visible in a terminal window. \n+The screen was also not centred in the room, which meant\n+that people in the front row sometimes couldn't see it around the instructor,\n+and there was a pillar in the room which didn't help visibility either.\n+The learners were pretty patient with this, but more than for most workshops\n+they really needed to have material on their own machines to be able to follow along.\n+Another issue with the room was that the desks were fairly close together: for\n+cases where a learner would have been best-served by working with a helper rather\n+than following the lesson, it wasn't really practical for them to move elsewhere,\n+and conversations with the helper couldn't be kept quiet enough to not disturb\n+others.\n+</p>\n+\n+<a href=\"http://swcarpentry.github.io/2014-10-30-utoronto/lessons/\">The\n+  material we covered</a> included: shell (P. Barmby), Git (G. Wilson), Python (P. Barmby and F. Xu), and SQL\n+(T. Guignard).</p>\n+<p>\n+<i>Pauline:</i>Most of the material in the beginner shell lessons was covered the first morning.\n+I found the pace a bit rushed and I would have liked more time for the students to \n+work on their own - I think the lessons contain too much material to really be covered\n+in 3 hours. Topics I skipped over or felt I didn't explain very well\n+included \"spaces in filenames\" and \"redirecting input\".\n+I didn't feel that I made very good use of the sticky note\n+system in the bash session, although I did try something else: I made the\n+questions at the end of the lessons into multiple-choice questions where they weren't\n+already, and asked the learners to vote on them with coloured cards (link).\n+I think this would have been useful except that the available projection screen\n+real estate was so small that the questions were unreadable!\n+</p>\n+\n+<p>\n+At the morning break, we put a tar file containing the \"Nelle\" file/directory structure\n+and a PDF file with the multiple-choice questions on Dropbox, and links to these\n+in the Etherpad. Learners were able to grab these and found them helpful in following along\n+for the remainder of the bash session. Feedback from the morning session\n+included the request to have these files available earlier, and to have an outline\n+of topics available before the session started, plus requests to go slower!\n+</p>\n+\n+<p>\n+The afternoon Git session went very smoothly. Greg took the learners through making\n+a repository and adding a remote on GitHub, the change/add/commit/push cycle, forking an existing \n+repository and adding the upstream remote, submitting pull requests, and \n+managing conflicts. As mentioned above, a few students ran into the problem with Git not \n+working on their Macs, for which the most obvious (but not-so-practical) fix is to \n+download Xcode and recompile <a\n+href=\"http://lists.software-carpentry.org/pipermail/discuss_lists.software-carpentry.org/2014-October/002315.html\">here</a>\n+is a link to the e-mail list thread on this topic.\n+<i>Greg:</i> ANYTHING TO ADD HERE?\n+</p>\n+\n+<i>Pauline:</i>\n+The second morning's Python session could have gone more smoothly. I used the\n+IPython console in Spyder, rather than the IPython notebook for this, in part because\n+I thought that the notebook would result in me moving too quickly and in part\n+because I've been having problems with notebooks scrolling very slowly on my machine.\n+I suggested to students that they should clone the workshop repo in order to\n+get the CSV and other files we'd be working with, and so that they could follow along\n+in the notebooks; or they could also just download specific files from","path":"blog/2014/11/toronto-gerstein-workshop.html","position":98,"original_position":98,"commit_id":"8a267c3672f39b67248ffc24a6a52f3de429e33a","original_commit_id":"8a267c3672f39b67248ffc24a6a52f3de429e33a","user":{"login":"lwjohnst86","id":6662983,"avatar_url":"https://avatars.githubusercontent.com/u/6662983?v=3","gravatar_id":"","url":"https://api.github.com/users/lwjohnst86","html_url":"https://github.com/lwjohnst86","followers_url":"https://api.github.com/users/lwjohnst86/followers","following_url":"https://api.github.com/users/lwjohnst86/following{/other_user}","gists_url":"https://api.github.com/users/lwjohnst86/gists{/gist_id}","starred_url":"https://api.github.com/users/lwjohnst86/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lwjohnst86/subscriptions","organizations_url":"https://api.github.com/users/lwjohnst86/orgs","repos_url":"https://api.github.com/users/lwjohnst86/repos","events_url":"https://api.github.com/users/lwjohnst86/events{/privacy}","received_events_url":"https://api.github.com/users/lwjohnst86/received_events","type":"User","site_admin":false},"body":"This was a big time waster. Many learners were still downloading the repo while the python lesson continued, so many were way behind and felt lost, having not even had a chance to even get the csv dataset loaded and interacting with it in python.","created_at":"2014-11-10T17:40:38Z","updated_at":"2014-11-10T17:40:38Z","html_url":"https://github.com/swcarpentry/site/pull/664#discussion_r20098145","pull_request_url":"https://api.github.com/repos/swcarpentry/site/pulls/664","_links":{"self":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/comments/20098145"},"html":{"href":"https://github.com/swcarpentry/site/pull/664#discussion_r20098145"},"pull_request":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/664"}}},"pull_request":{"url":"https://api.github.com/repos/swcarpentry/site/pulls/664","id":24105038,"html_url":"https://github.com/swcarpentry/site/pull/664","diff_url":"https://github.com/swcarpentry/site/pull/664.diff","patch_url":"https://github.com/swcarpentry/site/pull/664.patch","issue_url":"https://api.github.com/repos/swcarpentry/site/issues/664","number":664,"state":"open","locked":false,"title":"Blog post draft describing 2014-10-30-toronto workshop","user":{"login":"PBarmby","id":2228639,"avatar_url":"https://avatars.githubusercontent.com/u/2228639?v=3","gravatar_id":"","url":"https://api.github.com/users/PBarmby","html_url":"https://github.com/PBarmby","followers_url":"https://api.github.com/users/PBarmby/followers","following_url":"https://api.github.com/users/PBarmby/following{/other_user}","gists_url":"https://api.github.com/users/PBarmby/gists{/gist_id}","starred_url":"https://api.github.com/users/PBarmby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PBarmby/subscriptions","organizations_url":"https://api.github.com/users/PBarmby/orgs","repos_url":"https://api.github.com/users/PBarmby/repos","events_url":"https://api.github.com/users/PBarmby/events{/privacy}","received_events_url":"https://api.github.com/users/PBarmby/received_events","type":"User","site_admin":false},"body":"Needs contributions from other instructors and general checking..","created_at":"2014-11-08T19:20:04Z","updated_at":"2014-11-10T17:40:38Z","closed_at":null,"merged_at":null,"merge_commit_sha":"d6ad66b95a9f7c99efbef2bdaab27193d47b2d2f","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/swcarpentry/site/pulls/664/commits","review_comments_url":"https://api.github.com/repos/swcarpentry/site/pulls/664/comments","review_comment_url":"https://api.github.com/repos/swcarpentry/site/pulls/comments/{number}","comments_url":"https://api.github.com/repos/swcarpentry/site/issues/664/comments","statuses_url":"https://api.github.com/repos/swcarpentry/site/statuses/8a267c3672f39b67248ffc24a6a52f3de429e33a","head":{"label":"PBarmby:master","ref":"master","sha":"8a267c3672f39b67248ffc24a6a52f3de429e33a","user":{"login":"PBarmby","id":2228639,"avatar_url":"https://avatars.githubusercontent.com/u/2228639?v=3","gravatar_id":"","url":"https://api.github.com/users/PBarmby","html_url":"https://github.com/PBarmby","followers_url":"https://api.github.com/users/PBarmby/followers","following_url":"https://api.github.com/users/PBarmby/following{/other_user}","gists_url":"https://api.github.com/users/PBarmby/gists{/gist_id}","starred_url":"https://api.github.com/users/PBarmby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PBarmby/subscriptions","organizations_url":"https://api.github.com/users/PBarmby/orgs","repos_url":"https://api.github.com/users/PBarmby/repos","events_url":"https://api.github.com/users/PBarmby/events{/privacy}","received_events_url":"https://api.github.com/users/PBarmby/received_events","type":"User","site_admin":false},"repo":{"id":18183982,"name":"site","full_name":"PBarmby/site","owner":{"login":"PBarmby","id":2228639,"avatar_url":"https://avatars.githubusercontent.com/u/2228639?v=3","gravatar_id":"","url":"https://api.github.com/users/PBarmby","html_url":"https://github.com/PBarmby","followers_url":"https://api.github.com/users/PBarmby/followers","following_url":"https://api.github.com/users/PBarmby/following{/other_user}","gists_url":"https://api.github.com/users/PBarmby/gists{/gist_id}","starred_url":"https://api.github.com/users/PBarmby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PBarmby/subscriptions","organizations_url":"https://api.github.com/users/PBarmby/orgs","repos_url":"https://api.github.com/users/PBarmby/repos","events_url":"https://api.github.com/users/PBarmby/events{/privacy}","received_events_url":"https://api.github.com/users/PBarmby/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/PBarmby/site","description":"Software Carpentry web site.","fork":true,"url":"https://api.github.com/repos/PBarmby/site","forks_url":"https://api.github.com/repos/PBarmby/site/forks","keys_url":"https://api.github.com/repos/PBarmby/site/keys{/key_id}","collaborators_url":"https://api.github.com/repos/PBarmby/site/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/PBarmby/site/teams","hooks_url":"https://api.github.com/repos/PBarmby/site/hooks","issue_events_url":"https://api.github.com/repos/PBarmby/site/issues/events{/number}","events_url":"https://api.github.com/repos/PBarmby/site/events","assignees_url":"https://api.github.com/repos/PBarmby/site/assignees{/user}","branches_url":"https://api.github.com/repos/PBarmby/site/branches{/branch}","tags_url":"https://api.github.com/repos/PBarmby/site/tags","blobs_url":"https://api.github.com/repos/PBarmby/site/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/PBarmby/site/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/PBarmby/site/git/refs{/sha}","trees_url":"https://api.github.com/repos/PBarmby/site/git/trees{/sha}","statuses_url":"https://api.github.com/repos/PBarmby/site/statuses/{sha}","languages_url":"https://api.github.com/repos/PBarmby/site/languages","stargazers_url":"https://api.github.com/repos/PBarmby/site/stargazers","contributors_url":"https://api.github.com/repos/PBarmby/site/contributors","subscribers_url":"https://api.github.com/repos/PBarmby/site/subscribers","subscription_url":"https://api.github.com/repos/PBarmby/site/subscription","commits_url":"https://api.github.com/repos/PBarmby/site/commits{/sha}","git_commits_url":"https://api.github.com/repos/PBarmby/site/git/commits{/sha}","comments_url":"https://api.github.com/repos/PBarmby/site/comments{/number}","issue_comment_url":"https://api.github.com/repos/PBarmby/site/issues/comments/{number}","contents_url":"https://api.github.com/repos/PBarmby/site/contents/{+path}","compare_url":"https://api.github.com/repos/PBarmby/site/compare/{base}...{head}","merges_url":"https://api.github.com/repos/PBarmby/site/merges","archive_url":"https://api.github.com/repos/PBarmby/site/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/PBarmby/site/downloads","issues_url":"https://api.github.com/repos/PBarmby/site/issues{/number}","pulls_url":"https://api.github.com/repos/PBarmby/site/pulls{/number}","milestones_url":"https://api.github.com/repos/PBarmby/site/milestones{/number}","notifications_url":"https://api.github.com/repos/PBarmby/site/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/PBarmby/site/labels{/name}","releases_url":"https://api.github.com/repos/PBarmby/site/releases{/id}","created_at":"2014-03-27T17:03:38Z","updated_at":"2014-11-08T16:42:51Z","pushed_at":"2014-11-08T19:21:46Z","git_url":"git://github.com/PBarmby/site.git","ssh_url":"git@github.com:PBarmby/site.git","clone_url":"https://github.com/PBarmby/site.git","svn_url":"https://github.com/PBarmby/site","homepage":null,"size":71201,"stargazers_count":0,"watchers_count":0,"language":"TeX","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"swcarpentry:master","ref":"master","sha":"1216e1a3b1d0a2ef5ed65899b138064329171031","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":11396385,"name":"site","full_name":"swcarpentry/site","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/site","description":"Software Carpentry web site.","fork":false,"url":"https://api.github.com/repos/swcarpentry/site","forks_url":"https://api.github.com/repos/swcarpentry/site/forks","keys_url":"https://api.github.com/repos/swcarpentry/site/keys{/key_id}","collaborators_url":"https://api.github.com/repos/swcarpentry/site/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/swcarpentry/site/teams","hooks_url":"https://api.github.com/repos/swcarpentry/site/hooks","issue_events_url":"https://api.github.com/repos/swcarpentry/site/issues/events{/number}","events_url":"https://api.github.com/repos/swcarpentry/site/events","assignees_url":"https://api.github.com/repos/swcarpentry/site/assignees{/user}","branches_url":"https://api.github.com/repos/swcarpentry/site/branches{/branch}","tags_url":"https://api.github.com/repos/swcarpentry/site/tags","blobs_url":"https://api.github.com/repos/swcarpentry/site/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/swcarpentry/site/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/swcarpentry/site/git/refs{/sha}","trees_url":"https://api.github.com/repos/swcarpentry/site/git/trees{/sha}","statuses_url":"https://api.github.com/repos/swcarpentry/site/statuses/{sha}","languages_url":"https://api.github.com/repos/swcarpentry/site/languages","stargazers_url":"https://api.github.com/repos/swcarpentry/site/stargazers","contributors_url":"https://api.github.com/repos/swcarpentry/site/contributors","subscribers_url":"https://api.github.com/repos/swcarpentry/site/subscribers","subscription_url":"https://api.github.com/repos/swcarpentry/site/subscription","commits_url":"https://api.github.com/repos/swcarpentry/site/commits{/sha}","git_commits_url":"https://api.github.com/repos/swcarpentry/site/git/commits{/sha}","comments_url":"https://api.github.com/repos/swcarpentry/site/comments{/number}","issue_comment_url":"https://api.github.com/repos/swcarpentry/site/issues/comments/{number}","contents_url":"https://api.github.com/repos/swcarpentry/site/contents/{+path}","compare_url":"https://api.github.com/repos/swcarpentry/site/compare/{base}...{head}","merges_url":"https://api.github.com/repos/swcarpentry/site/merges","archive_url":"https://api.github.com/repos/swcarpentry/site/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/swcarpentry/site/downloads","issues_url":"https://api.github.com/repos/swcarpentry/site/issues{/number}","pulls_url":"https://api.github.com/repos/swcarpentry/site/pulls{/number}","milestones_url":"https://api.github.com/repos/swcarpentry/site/milestones{/number}","notifications_url":"https://api.github.com/repos/swcarpentry/site/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/swcarpentry/site/labels{/name}","releases_url":"https://api.github.com/repos/swcarpentry/site/releases{/id}","created_at":"2013-07-14T00:49:11Z","updated_at":"2014-10-26T07:00:02Z","pushed_at":"2014-11-10T02:13:25Z","git_url":"git://github.com/swcarpentry/site.git","ssh_url":"git@github.com:swcarpentry/site.git","clone_url":"https://github.com/swcarpentry/site.git","svn_url":"https://github.com/swcarpentry/site","homepage":null,"size":90916,"stargazers_count":26,"watchers_count":26,"language":"TeX","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":73,"mirror_url":null,"open_issues_count":16,"forks":73,"open_issues":16,"watchers":26,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/664"},"html":{"href":"https://github.com/swcarpentry/site/pull/664"},"issue":{"href":"https://api.github.com/repos/swcarpentry/site/issues/664"},"comments":{"href":"https://api.github.com/repos/swcarpentry/site/issues/664/comments"},"review_comments":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/664/comments"},"review_comment":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/swcarpentry/site/pulls/664/commits"},"statuses":{"href":"https://api.github.com/repos/swcarpentry/site/statuses/8a267c3672f39b67248ffc24a6a52f3de429e33a"}}}}