{"action":"created","issue":{"url":"https://api.github.com/repos/dpinney/omf/issues/279","labels_url":"https://api.github.com/repos/dpinney/omf/issues/279/labels{/name}","comments_url":"https://api.github.com/repos/dpinney/omf/issues/279/comments","events_url":"https://api.github.com/repos/dpinney/omf/issues/279/events","html_url":"https://github.com/dpinney/omf/issues/279","id":43969080,"number":279,"title":"Improved location indicator for solarFinancial.","user":{"login":"dpinney","id":2131438,"avatar_url":"https://avatars.githubusercontent.com/u/2131438?v=3","gravatar_id":"","url":"https://api.github.com/users/dpinney","html_url":"https://github.com/dpinney","followers_url":"https://api.github.com/users/dpinney/followers","following_url":"https://api.github.com/users/dpinney/following{/other_user}","gists_url":"https://api.github.com/users/dpinney/gists{/gist_id}","starred_url":"https://api.github.com/users/dpinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpinney/subscriptions","organizations_url":"https://api.github.com/users/dpinney/orgs","repos_url":"https://api.github.com/users/dpinney/repos","events_url":"https://api.github.com/users/dpinney/events{/privacy}","received_events_url":"https://api.github.com/users/dpinney/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"mannanj","id":7212777,"avatar_url":"https://avatars.githubusercontent.com/u/7212777?v=3","gravatar_id":"","url":"https://api.github.com/users/mannanj","html_url":"https://github.com/mannanj","followers_url":"https://api.github.com/users/mannanj/followers","following_url":"https://api.github.com/users/mannanj/following{/other_user}","gists_url":"https://api.github.com/users/mannanj/gists{/gist_id}","starred_url":"https://api.github.com/users/mannanj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mannanj/subscriptions","organizations_url":"https://api.github.com/users/mannanj/orgs","repos_url":"https://api.github.com/users/mannanj/repos","events_url":"https://api.github.com/users/mannanj/events{/privacy}","received_events_url":"https://api.github.com/users/mannanj/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/dpinney/omf/milestones/5","html_url":"https://github.com/dpinney/omf/milestones/The%20Future","labels_url":"https://api.github.com/repos/dpinney/omf/milestones/5/labels","id":327854,"number":5,"title":"The Future","description":"Unscheduled issues.","creator":{"login":"dpinney","id":2131438,"avatar_url":"https://avatars.githubusercontent.com/u/2131438?v=3","gravatar_id":"","url":"https://api.github.com/users/dpinney","html_url":"https://github.com/dpinney","followers_url":"https://api.github.com/users/dpinney/followers","following_url":"https://api.github.com/users/dpinney/following{/other_user}","gists_url":"https://api.github.com/users/dpinney/gists{/gist_id}","starred_url":"https://api.github.com/users/dpinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpinney/subscriptions","organizations_url":"https://api.github.com/users/dpinney/orgs","repos_url":"https://api.github.com/users/dpinney/repos","events_url":"https://api.github.com/users/dpinney/events{/privacy}","received_events_url":"https://api.github.com/users/dpinney/received_events","type":"User","site_admin":false},"open_issues":47,"closed_issues":64,"state":"open","created_at":"2013-05-06T15:48:03Z","updated_at":"2015-06-17T21:22:09Z","due_on":"2100-01-01T08:00:00Z","closed_at":null},"comments":4,"created_at":"2014-09-25T21:20:14Z","updated_at":"2015-06-18T18:39:58Z","closed_at":null,"body":"TODO\r\n* Move the zipcode mapping stuff to weather.py.\r\n* Add the zip code logic to the other solar models.\r\n\r\nQUESTIONS\r\n* Currently users can only specify a solar model location by choosing a TMY file. Ideally, we want to allow them to choose either a zipcode or a lat/lon pair or a TMY location."},"comment":{"url":"https://api.github.com/repos/dpinney/omf/issues/comments/113253348","html_url":"https://github.com/dpinney/omf/issues/279#issuecomment-113253348","issue_url":"https://api.github.com/repos/dpinney/omf/issues/279","id":113253348,"user":{"login":"dpinney","id":2131438,"avatar_url":"https://avatars.githubusercontent.com/u/2131438?v=3","gravatar_id":"","url":"https://api.github.com/users/dpinney","html_url":"https://github.com/dpinney","followers_url":"https://api.github.com/users/dpinney/followers","following_url":"https://api.github.com/users/dpinney/following{/other_user}","gists_url":"https://api.github.com/users/dpinney/gists{/gist_id}","starred_url":"https://api.github.com/users/dpinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpinney/subscriptions","organizations_url":"https://api.github.com/users/dpinney/orgs","repos_url":"https://api.github.com/users/dpinney/repos","events_url":"https://api.github.com/users/dpinney/events{/privacy}","received_events_url":"https://api.github.com/users/dpinney/received_events","type":"User","site_admin":false},"created_at":"2015-06-18T18:39:58Z","updated_at":"2015-06-18T18:39:58Z","body":"Code looks pretty good. We need a test for zipCodeToClimateName. Also, imports need to go at the top of the file and not in the function definition. It's a python convention."}}