{"comment":{"url":"https://api.github.com/repos/young-asap/DSI-SF-4-young-asap/comments/20149578","html_url":"https://github.com/young-asap/DSI-SF-4-young-asap/commit/12725f47aae3f095c2f8d9acbdb0a5aea48ead14#commitcomment-20149578","id":20149578,"user":{"login":"rddunlap23","id":10054573,"avatar_url":"https://avatars.githubusercontent.com/u/10054573?v=3","gravatar_id":"","url":"https://api.github.com/users/rddunlap23","html_url":"https://github.com/rddunlap23","followers_url":"https://api.github.com/users/rddunlap23/followers","following_url":"https://api.github.com/users/rddunlap23/following{/other_user}","gists_url":"https://api.github.com/users/rddunlap23/gists{/gist_id}","starred_url":"https://api.github.com/users/rddunlap23/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rddunlap23/subscriptions","organizations_url":"https://api.github.com/users/rddunlap23/orgs","repos_url":"https://api.github.com/users/rddunlap23/repos","events_url":"https://api.github.com/users/rddunlap23/events{/privacy}","received_events_url":"https://api.github.com/users/rddunlap23/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":null,"commit_id":"12725f47aae3f095c2f8d9acbdb0a5aea48ead14","created_at":"2016-12-12T01:01:26Z","updated_at":"2016-12-12T01:01:26Z","body":"Incomplete (0) Does Not Meet Expectations (1) Meets Expectations (2) Exceeds Expectations (3)\r\n\r\n1) 3\r\n2.1) 2 (Labels!)\r\n2.2) 2 (Want to add + 1 for ln to avoid ln(0). Granted not a big issue for this data set...)\r\n3.1) 2\r\n3.2) 3\r\n3.3) 2\r\n3.4) 2\r\n4.1) 3\r\n4.2) 1 (For features that are missing most values makes sense to drop those. For those that only have a small subset missing I would think through if you can impute the missing values so that you're not throwing away a bunch of valuable data)\r\n4.3)1 (See solutions/comments above)\r\n5) 2 (You can rotate X axis labels to make more readable)\r\n6) 3\r\n7.1) 3\r\n7.2) 1 (Missing the explanations!)\r\n8.1) 2 (How could have used dtypes and list comprehension to get column headers out quickly?)\r\n8.2) 3 \r\n8.3) 1 (Missing the plot)\r\n8.4) 2\r\n8.5) 2 (If you sort it really makes the chart pop)\r\n9.1) 2 (That much typing makes me sad. See solutions for quicker solve)\r\n9.2) 2\r\n9.3) 2\r\n10.1) 3\r\n10.2) 2\r\n10.3) 3\r\n11.1) 2       \r\n11.2) 1\r\n11.3) 0\r\n11.4) 0\r\n12.1) 0\r\n12.2) 0\r\n12.3) 0\r\n12.4) 0\r\n13) 0\r\n14) 0\r\n15) 0\r\n\r\nTotal  57/108\r\n\r\n**Things that worked really well:**\r\n- Overall really solid work. \r\n- Great job on the bonus questions you answered. \r\n\r\n**Suggestions**\r\n- Don't forget to put axis labels and titles on plots. \r\n- See solutions for tricks on creating X matrix more efficiently\r\n- Check solutions for last few problems and make sure you're comfortable with these. "}}
{"actor":{"display_login":"rddunlap23"}}