{"action":"created","comment":{"url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/comments/28562794","id":28562794,"diff_hunk":"@@ -32,17 +36,12 @@ def sum_orders(customers, orders):\n     \"\"\"\n     combined_dict = {}\n     for order in orders.itervalues():\n-        cust_id = order['customer_id']\n-        cust_name = customers[cust_id]['name']\n-        cust_email = customers[cust_id]['email']\n-        if cust_id not in combined_dict and customers.keys():\n-            counter = 1\n-            t_orders = order['total']\n+        if order['customer_id'] not in combined_dict:\n+            combined_dict[order['customer_id']] = {'total': order['total']}\n+            combined_dict[order['customer_id']]['orders'] = 1\n+            combined_dict[order['customer_id']].update(customers\n+                                                       [order['customer_id']])\n         else:\n-            combined_dict[cust_id]['orders'] += 1\n-            t_orders = ((combined_dict[cust_id]['total']) + (order['total']))\n-        final_order = dict(name=cust_name, email=cust_email, orders=counter,\n-                           total=t_orders)\n-        cust_final_order = {cust_id: final_order}\n-        combined_dict.update(cust_final_order)\n-    return combined_dict\n+            combined_dict[order['customer_id']]['total'] += order['total']\n+            combined_dict[order['customer_id']]['orders'] += 1\n+            return combined_dict","path":"task_01.py","position":46,"original_position":46,"commit_id":"997e06b70a48cd7730e5b876835e5fb492bdc88b","original_commit_id":"997e06b70a48cd7730e5b876835e5fb492bdc88b","user":{"login":"eliz79","id":10748900,"avatar_url":"https://avatars.githubusercontent.com/u/10748900?v=3","gravatar_id":"","url":"https://api.github.com/users/eliz79","html_url":"https://github.com/eliz79","followers_url":"https://api.github.com/users/eliz79/followers","following_url":"https://api.github.com/users/eliz79/following{/other_user}","gists_url":"https://api.github.com/users/eliz79/gists{/gist_id}","starred_url":"https://api.github.com/users/eliz79/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eliz79/subscriptions","organizations_url":"https://api.github.com/users/eliz79/orgs","repos_url":"https://api.github.com/users/eliz79/repos","events_url":"https://api.github.com/users/eliz79/events{/privacy}","received_events_url":"https://api.github.com/users/eliz79/received_events","type":"User","site_admin":false},"body":"i did and seems fine but still same error failing","created_at":"2015-04-17T00:39:45Z","updated_at":"2015-04-17T00:39:45Z","html_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13#discussion_r28562794","pull_request_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13","_links":{"self":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/comments/28562794"},"html":{"href":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13#discussion_r28562794"},"pull_request":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13"}}},"pull_request":{"url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13","id":33222186,"html_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13","diff_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13.diff","patch_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13.patch","issue_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/13","number":13,"state":"open","locked":false,"title":"updating task 1","user":{"login":"eliz79","id":10748900,"avatar_url":"https://avatars.githubusercontent.com/u/10748900?v=3","gravatar_id":"","url":"https://api.github.com/users/eliz79","html_url":"https://github.com/eliz79","followers_url":"https://api.github.com/users/eliz79/followers","following_url":"https://api.github.com/users/eliz79/following{/other_user}","gists_url":"https://api.github.com/users/eliz79/gists{/gist_id}","starred_url":"https://api.github.com/users/eliz79/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eliz79/subscriptions","organizations_url":"https://api.github.com/users/eliz79/orgs","repos_url":"https://api.github.com/users/eliz79/repos","events_url":"https://api.github.com/users/eliz79/events{/privacy}","received_events_url":"https://api.github.com/users/eliz79/received_events","type":"User","site_admin":false},"body":"I tried to eliminate as much as i could of unnecessary variables. I know im close but quite cant get on adding customer 3. Ill work on it again tomorrow for a bit. Hopefully i can get closer. ","created_at":"2015-04-14T02:44:49Z","updated_at":"2015-04-17T00:39:45Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5ef4dd0df3b1bb005733e1991d902077e68a54aa","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13/commits","review_comments_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13/comments","review_comment_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/comments{/number}","comments_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/13/comments","statuses_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/statuses/997e06b70a48cd7730e5b876835e5fb492bdc88b","head":{"label":"eliz79:master","ref":"master","sha":"997e06b70a48cd7730e5b876835e5fb492bdc88b","user":{"login":"eliz79","id":10748900,"avatar_url":"https://avatars.githubusercontent.com/u/10748900?v=3","gravatar_id":"","url":"https://api.github.com/users/eliz79","html_url":"https://github.com/eliz79","followers_url":"https://api.github.com/users/eliz79/followers","following_url":"https://api.github.com/users/eliz79/following{/other_user}","gists_url":"https://api.github.com/users/eliz79/gists{/gist_id}","starred_url":"https://api.github.com/users/eliz79/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eliz79/subscriptions","organizations_url":"https://api.github.com/users/eliz79/orgs","repos_url":"https://api.github.com/users/eliz79/repos","events_url":"https://api.github.com/users/eliz79/events{/privacy}","received_events_url":"https://api.github.com/users/eliz79/received_events","type":"User","site_admin":false},"repo":{"id":33628510,"name":"is210-week-10-synthesizing","full_name":"eliz79/is210-week-10-synthesizing","owner":{"login":"eliz79","id":10748900,"avatar_url":"https://avatars.githubusercontent.com/u/10748900?v=3","gravatar_id":"","url":"https://api.github.com/users/eliz79","html_url":"https://github.com/eliz79","followers_url":"https://api.github.com/users/eliz79/followers","following_url":"https://api.github.com/users/eliz79/following{/other_user}","gists_url":"https://api.github.com/users/eliz79/gists{/gist_id}","starred_url":"https://api.github.com/users/eliz79/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eliz79/subscriptions","organizations_url":"https://api.github.com/users/eliz79/orgs","repos_url":"https://api.github.com/users/eliz79/repos","events_url":"https://api.github.com/users/eliz79/events{/privacy}","received_events_url":"https://api.github.com/users/eliz79/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/eliz79/is210-week-10-synthesizing","description":"Week #10 Synthesizing Tasks","fork":true,"url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing","forks_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/forks","keys_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/keys{/key_id}","collaborators_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/teams","hooks_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/hooks","issue_events_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/issues/events{/number}","events_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/events","assignees_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/assignees{/user}","branches_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/branches{/branch}","tags_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/tags","blobs_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/git/refs{/sha}","trees_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/git/trees{/sha}","statuses_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/statuses/{sha}","languages_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/languages","stargazers_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/stargazers","contributors_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/contributors","subscribers_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/subscribers","subscription_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/subscription","commits_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/commits{/sha}","git_commits_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/git/commits{/sha}","comments_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/comments{/number}","issue_comment_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/issues/comments{/number}","contents_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/contents/{+path}","compare_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/compare/{base}...{head}","merges_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/merges","archive_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/downloads","issues_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/issues{/number}","pulls_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/pulls{/number}","milestones_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/milestones{/number}","notifications_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/labels{/name}","releases_url":"https://api.github.com/repos/eliz79/is210-week-10-synthesizing/releases{/id}","created_at":"2015-04-08T19:47:05Z","updated_at":"2015-04-17T00:07:17Z","pushed_at":"2015-04-17T00:07:17Z","git_url":"git://github.com/eliz79/is210-week-10-synthesizing.git","ssh_url":"git@github.com:eliz79/is210-week-10-synthesizing.git","clone_url":"https://github.com/eliz79/is210-week-10-synthesizing.git","svn_url":"https://github.com/eliz79/is210-week-10-synthesizing","homepage":null,"size":105,"stargazers_count":0,"watchers_count":0,"language":"Python","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":"is210-2015-spring-01:eliz79","ref":"eliz79","sha":"0c6c3a477eb186de9cad22ed5ed3704729314289","user":{"login":"is210-2015-spring-01","id":10734022,"avatar_url":"https://avatars.githubusercontent.com/u/10734022?v=3","gravatar_id":"","url":"https://api.github.com/users/is210-2015-spring-01","html_url":"https://github.com/is210-2015-spring-01","followers_url":"https://api.github.com/users/is210-2015-spring-01/followers","following_url":"https://api.github.com/users/is210-2015-spring-01/following{/other_user}","gists_url":"https://api.github.com/users/is210-2015-spring-01/gists{/gist_id}","starred_url":"https://api.github.com/users/is210-2015-spring-01/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/is210-2015-spring-01/subscriptions","organizations_url":"https://api.github.com/users/is210-2015-spring-01/orgs","repos_url":"https://api.github.com/users/is210-2015-spring-01/repos","events_url":"https://api.github.com/users/is210-2015-spring-01/events{/privacy}","received_events_url":"https://api.github.com/users/is210-2015-spring-01/received_events","type":"Organization","site_admin":false},"repo":{"id":33101749,"name":"is210-week-10-synthesizing","full_name":"is210-2015-spring-01/is210-week-10-synthesizing","owner":{"login":"is210-2015-spring-01","id":10734022,"avatar_url":"https://avatars.githubusercontent.com/u/10734022?v=3","gravatar_id":"","url":"https://api.github.com/users/is210-2015-spring-01","html_url":"https://github.com/is210-2015-spring-01","followers_url":"https://api.github.com/users/is210-2015-spring-01/followers","following_url":"https://api.github.com/users/is210-2015-spring-01/following{/other_user}","gists_url":"https://api.github.com/users/is210-2015-spring-01/gists{/gist_id}","starred_url":"https://api.github.com/users/is210-2015-spring-01/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/is210-2015-spring-01/subscriptions","organizations_url":"https://api.github.com/users/is210-2015-spring-01/orgs","repos_url":"https://api.github.com/users/is210-2015-spring-01/repos","events_url":"https://api.github.com/users/is210-2015-spring-01/events{/privacy}","received_events_url":"https://api.github.com/users/is210-2015-spring-01/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing","description":"Week #10 Synthesizing Tasks","fork":true,"url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing","forks_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/forks","keys_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/keys{/key_id}","collaborators_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/teams","hooks_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/hooks","issue_events_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/events{/number}","events_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/events","assignees_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/assignees{/user}","branches_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/branches{/branch}","tags_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/tags","blobs_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/git/refs{/sha}","trees_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/git/trees{/sha}","statuses_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/statuses/{sha}","languages_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/languages","stargazers_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/stargazers","contributors_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/contributors","subscribers_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/subscribers","subscription_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/subscription","commits_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/commits{/sha}","git_commits_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/git/commits{/sha}","comments_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/comments{/number}","issue_comment_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/comments{/number}","contents_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/contents/{+path}","compare_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/compare/{base}...{head}","merges_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/merges","archive_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/downloads","issues_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues{/number}","pulls_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls{/number}","milestones_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/milestones{/number}","notifications_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/labels{/name}","releases_url":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/releases{/id}","created_at":"2015-03-30T03:40:46Z","updated_at":"2015-04-08T18:38:26Z","pushed_at":"2015-04-17T00:07:17Z","git_url":"git://github.com/is210-2015-spring-01/is210-week-10-synthesizing.git","ssh_url":"git@github.com:is210-2015-spring-01/is210-week-10-synthesizing.git","clone_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing.git","svn_url":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing","homepage":null,"size":1338,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":13,"mirror_url":null,"open_issues_count":3,"forks":13,"open_issues":3,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13"},"html":{"href":"https://github.com/is210-2015-spring-01/is210-week-10-synthesizing/pull/13"},"issue":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/13"},"comments":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/issues/13/comments"},"review_comments":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13/comments"},"review_comment":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/pulls/13/commits"},"statuses":{"href":"https://api.github.com/repos/is210-2015-spring-01/is210-week-10-synthesizing/statuses/997e06b70a48cd7730e5b876835e5fb492bdc88b"}}}}