{"action":"closed","issue":{"url":"https://api.github.com/repos/berkeley-stat159/project-epsilon/issues/44","labels_url":"https://api.github.com/repos/berkeley-stat159/project-epsilon/issues/44/labels{/name}","comments_url":"https://api.github.com/repos/berkeley-stat159/project-epsilon/issues/44/comments","events_url":"https://api.github.com/repos/berkeley-stat159/project-epsilon/issues/44/events","html_url":"https://github.com/berkeley-stat159/project-epsilon/issues/44","id":117403364,"number":44,"title":"feedback","user":{"login":"jarrodmillman","id":123428,"avatar_url":"https://avatars.githubusercontent.com/u/123428?v=3","gravatar_id":"","url":"https://api.github.com/users/jarrodmillman","html_url":"https://github.com/jarrodmillman","followers_url":"https://api.github.com/users/jarrodmillman/followers","following_url":"https://api.github.com/users/jarrodmillman/following{/other_user}","gists_url":"https://api.github.com/users/jarrodmillman/gists{/gist_id}","starred_url":"https://api.github.com/users/jarrodmillman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jarrodmillman/subscriptions","organizations_url":"https://api.github.com/users/jarrodmillman/orgs","repos_url":"https://api.github.com/users/jarrodmillman/repos","events_url":"https://api.github.com/users/jarrodmillman/events{/privacy}","received_events_url":"https://api.github.com/users/jarrodmillman/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-11-17T17:15:16Z","updated_at":"2015-12-09T20:34:24Z","closed_at":"2015-12-09T20:34:24Z","body":"Good, clear description of the loss-aversion paper.\r\n\r\nGreat work with the initial regressor analysis; I liked to see the results of\r\nthe preliminary regressor applied to the data - represents real, tangible\r\nprogress toward an ultimate goal.\r\n\r\nClear presentation of the goal to experiment with the HRF model - was less clear\r\nabout where the motivation to do so came from; might want to justify the\r\ndecision to investigate this aspect a bit more.  For example, there is a published\r\nliterature addressing this question to some extent.  You don't need to replicate\r\nthe state of the art here, but you should reference existing work as part of\r\nyour motivation.\r\n\r\nSeemed to have a very solid understanding of what they knew about the data, and\r\nwhat they still needed to understand (PTvals for instance). Also have clearly\r\ndone literature review outside of their paper to help with the coding aspect\r\n(PEP8).\r\n\r\nThe group seems clearly capable and invested in the project, just might want to\r\nfocus on specific tasks a bit more.\r\n\r\nYou should make sure your code is valid.  For instance,\r\n\r\n    $ python eda_behav.py\r\n      File \"eda_behav.py\", line 3\r\n        %matplotlib\r\n        ^\r\n    SyntaxError: invalid syntax\r\n\r\nYou still have a lot of work to do, but there is still time.\r\n"}}