{"id":827,"date":"2009-10-06T16:09:32","date_gmt":"2009-10-06T21:09:32","guid":{"rendered":"http:\/\/mikeconley.ca\/blog\/?p=827"},"modified":"2023-12-20T16:25:18","modified_gmt":"2023-12-20T21:25:18","slug":"research-question-idea-3","status":"publish","type":"post","link":"https:\/\/mikeconley.ca\/blog\/2009\/10\/06\/research-question-idea-3\/","title":{"rendered":"Research Question Idea #3"},"content":{"rendered":"<p>When we started using ReviewBoard with <a href=\"http:\/\/www.markusproject.org\">MarkUs<\/a> a few months back, all of a sudden, commits to the repository seemed to slow down: we would take more time cleaning up our code, and polishing it for others to see.<\/p>\n<p>Our commits were usually quite large too.\u00a0 This is because we were all working on different sections of the code, and we wanted to commit stuff that &#8220;instantly worked&#8221; and was &#8220;instantly perfect&#8221;.\u00a0 So after days of silence, 1000 lines of code would suddenly go up for review&#8230;and as <a href=\"http:\/\/blog.asmartbear.com\/\">Jason Cohen<\/a> can probably tell you, the number of defects found during review decreases as the amount of code to look at increases.\u00a0 So, the reviewer would skip through 1000 lines, assume most of it was OK, and give it the Ship It.<\/p>\n<p>Yeah, I know.\u00a0 Awful.\u00a0 I wonder if this is a standard newbie mistake for student groups just starting out with code review&#8230;<\/p>\n<h3>So, study idea:<\/h3>\n<p>Have two separate groups working on some assignment.\u00a0 Have Group 1 commit to their repository without any review process.\u00a0 Have Group 2 do pre-commit reviews using a tool like <a href=\"http:\/\/www.review-board.org\/\">ReviewBoard<\/a>.<\/p>\n<p>Now check out the size, frequency, and readability of the repository diffs of each group.\u00a0 Might generate some interesting data.<\/p>\n<p>Anyhow, in our defence, we seem to have calmed down on MarkUs.\u00a0 Diffs up for review are pretty small, and get posted relatively frequently.\u00a0 Using ReviewBoard on MarkUs has made me a believer.\u00a0 Testify!<\/p>\n","protected":false},"excerpt":{"rendered":"<p>When we started using ReviewBoard with MarkUs a few months back, all of a sudden, commits to the repository seemed to slow down: we would take more time cleaning up our code, and polishing it for others to see. Our commits were usually quite large too.\u00a0 This is because we were all working on different [&hellip;]<\/p>\n","protected":false},"author":4,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"jetpack_post_was_ever_published":false,"_jetpack_newsletter_access":"","_jetpack_dont_email_post_to_subs":false,"_jetpack_newsletter_tier_id":0,"_jetpack_memberships_contains_paywalled_content":false,"_jetpack_memberships_contains_paid_content":false,"footnotes":"","jetpack_publicize_message":"","jetpack_publicize_feature_enabled":true,"jetpack_social_post_already_shared":false,"jetpack_social_options":{"image_generator_settings":{"template":"highway","default_image_id":0,"font":"","enabled":false},"version":2}},"categories":[5],"tags":[501,177,503,504,470],"class_list":["post-827","post","type-post","status-publish","format-standard","hentry","category-computer-science","tag-code-review","tag-grad-school","tag-research","tag-reviewboard","tag-study"],"jetpack_publicize_connections":[],"jetpack_featured_media_url":"","jetpack_shortlink":"https:\/\/wp.me\/prmTy-dl","jetpack_sharing_enabled":true,"_links":{"self":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/827","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/users\/4"}],"replies":[{"embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/comments?post=827"}],"version-history":[{"count":4,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/827\/revisions"}],"predecessor-version":[{"id":3218,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/827\/revisions\/3218"}],"wp:attachment":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/media?parent=827"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/categories?post=827"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/tags?post=827"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}