{"action":"created","issue":{"url":"https://api.github.com/repos/sympy/sympy/issues/9830","labels_url":"https://api.github.com/repos/sympy/sympy/issues/9830/labels{/name}","comments_url":"https://api.github.com/repos/sympy/sympy/issues/9830/comments","events_url":"https://api.github.com/repos/sympy/sympy/issues/9830/events","html_url":"https://github.com/sympy/sympy/pull/9830","id":101123099,"number":9830,"title":"Bug Fixed : Matrix power of identity matrix fails #9823","user":{"login":"kevalds51","id":9724752,"avatar_url":"https://avatars.githubusercontent.com/u/9724752?v=3","gravatar_id":"","url":"https://api.github.com/users/kevalds51","html_url":"https://github.com/kevalds51","followers_url":"https://api.github.com/users/kevalds51/followers","following_url":"https://api.github.com/users/kevalds51/following{/other_user}","gists_url":"https://api.github.com/users/kevalds51/gists{/gist_id}","starred_url":"https://api.github.com/users/kevalds51/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevalds51/subscriptions","organizations_url":"https://api.github.com/users/kevalds51/orgs","repos_url":"https://api.github.com/users/kevalds51/repos","events_url":"https://api.github.com/users/kevalds51/events{/privacy}","received_events_url":"https://api.github.com/users/kevalds51/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2015-08-14T23:48:12Z","updated_at":"2015-08-24T14:51:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/sympy/sympy/pulls/9830","html_url":"https://github.com/sympy/sympy/pull/9830","diff_url":"https://github.com/sympy/sympy/pull/9830.diff","patch_url":"https://github.com/sympy/sympy/pull/9830.patch"},"body":"Added the test to exercise the code change. (as requested by the @moorepants )\r\nModified the code such that no error is generated for both MatPow method and ** operator.\r\nremoved extra blank line which resulted in test_code_quality fail."},"comment":{"url":"https://api.github.com/repos/sympy/sympy/issues/comments/134232262","html_url":"https://github.com/sympy/sympy/pull/9830#issuecomment-134232262","issue_url":"https://api.github.com/repos/sympy/sympy/issues/9830","id":134232262,"user":{"login":"oliverlee","id":2965031,"avatar_url":"https://avatars.githubusercontent.com/u/2965031?v=3","gravatar_id":"","url":"https://api.github.com/users/oliverlee","html_url":"https://github.com/oliverlee","followers_url":"https://api.github.com/users/oliverlee/followers","following_url":"https://api.github.com/users/oliverlee/following{/other_user}","gists_url":"https://api.github.com/users/oliverlee/gists{/gist_id}","starred_url":"https://api.github.com/users/oliverlee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oliverlee/subscriptions","organizations_url":"https://api.github.com/users/oliverlee/orgs","repos_url":"https://api.github.com/users/oliverlee/repos","events_url":"https://api.github.com/users/oliverlee/events{/privacy}","received_events_url":"https://api.github.com/users/oliverlee/received_events","type":"User","site_admin":false},"created_at":"2015-08-24T14:51:48Z","updated_at":"2015-08-24T14:51:48Z","body":"* The use of ```as_explicit()``` in your ```ZeroMatrix``` and ```Identity``` tests seem unnecessary; positive powers should return the same class.\r\n* I don't think the result of taking an ```ImmutableMatrix``` to the zero power should be a different class but maybe someone else who is more familiar with the matrix stuff can weigh in.\r\n* I'm not sure how errors should be handled with regards to the ```ZeroMatrix``` taken to negative powers. Maybe a separate issue should be opened for that?\r\n\r\nOther than that, looks fine to me."}}