{"action":"created","issue":{"url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/1","repository_url":"https://api.github.com/repos/hackingmaterials/MatMiner","labels_url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/1/comments","events_url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/1/events","html_url":"https://github.com/hackingmaterials/MatMiner/pull/1","id":188619993,"number":1,"title":"A few fixes to CitrineDataRetrieval and added unit tests for retrieval","user":{"login":"montoyjh","id":6494516,"avatar_url":"https://avatars.githubusercontent.com/u/6494516?v=3","gravatar_id":"","url":"https://api.github.com/users/montoyjh","html_url":"https://github.com/montoyjh","followers_url":"https://api.github.com/users/montoyjh/followers","following_url":"https://api.github.com/users/montoyjh/following{/other_user}","gists_url":"https://api.github.com/users/montoyjh/gists{/gist_id}","starred_url":"https://api.github.com/users/montoyjh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/montoyjh/subscriptions","organizations_url":"https://api.github.com/users/montoyjh/orgs","repos_url":"https://api.github.com/users/montoyjh/repos","events_url":"https://api.github.com/users/montoyjh/events{/privacy}","received_events_url":"https://api.github.com/users/montoyjh/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-11-10T21:13:46Z","updated_at":"2016-11-11T19:50:42Z","closed_at":"2016-11-11T19:50:30Z","pull_request":{"url":"https://api.github.com/repos/hackingmaterials/MatMiner/pulls/1","html_url":"https://github.com/hackingmaterials/MatMiner/pull/1","diff_url":"https://github.com/hackingmaterials/MatMiner/pull/1.diff","patch_url":"https://github.com/hackingmaterials/MatMiner/pull/1.patch"},"body":"This PR fixes a few issues I was having with citrine data retrieval.  I went ahead and added a (very basic) unit test for CitrineDataRetrieval and MPDataRetrieval, following the convention in the MPRester unit test in which they are skipped without proper api credentials in the environment.\r\n\r\nAlso, as a side note, I had a bit of trouble with the citrine python API.  The PyPI versions of citrination-client do not work properly with MatMiner (nor does the master branch in their github repo).  I think users can figure it out by following the instructions at the link in the CitrineDataRetrieval docstring, but I figured I'd note it here just so it was on the radar."},"comment":{"url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/comments/260040726","html_url":"https://github.com/hackingmaterials/MatMiner/pull/1#issuecomment-260040726","issue_url":"https://api.github.com/repos/hackingmaterials/MatMiner/issues/1","id":260040726,"user":{"login":"saurabh02","id":8379761,"avatar_url":"https://avatars.githubusercontent.com/u/8379761?v=3","gravatar_id":"","url":"https://api.github.com/users/saurabh02","html_url":"https://github.com/saurabh02","followers_url":"https://api.github.com/users/saurabh02/followers","following_url":"https://api.github.com/users/saurabh02/following{/other_user}","gists_url":"https://api.github.com/users/saurabh02/gists{/gist_id}","starred_url":"https://api.github.com/users/saurabh02/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saurabh02/subscriptions","organizations_url":"https://api.github.com/users/saurabh02/orgs","repos_url":"https://api.github.com/users/saurabh02/repos","events_url":"https://api.github.com/users/saurabh02/events{/privacy}","received_events_url":"https://api.github.com/users/saurabh02/received_events","type":"User","site_admin":false},"created_at":"2016-11-11T19:50:42Z","updated_at":"2016-11-11T19:50:42Z","body":"Thanks @montoyjh!"}}
{"actor":{"display_login":"saurabh02"}}