{"action":"created","issue":{"url":"https://api.github.com/repos/wbhart/flint2/issues/115","labels_url":"https://api.github.com/repos/wbhart/flint2/issues/115/labels{/name}","comments_url":"https://api.github.com/repos/wbhart/flint2/issues/115/comments","events_url":"https://api.github.com/repos/wbhart/flint2/issues/115/events","html_url":"https://github.com/wbhart/flint2/pull/115","id":56175639,"number":115,"title":"Added code for perfect power testing in fmpz_factor.","user":{"login":"kush789","id":7972943,"avatar_url":"https://avatars.githubusercontent.com/u/7972943?v=3","gravatar_id":"","url":"https://api.github.com/users/kush789","html_url":"https://github.com/kush789","followers_url":"https://api.github.com/users/kush789/followers","following_url":"https://api.github.com/users/kush789/following{/other_user}","gists_url":"https://api.github.com/users/kush789/gists{/gist_id}","starred_url":"https://api.github.com/users/kush789/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kush789/subscriptions","organizations_url":"https://api.github.com/users/kush789/orgs","repos_url":"https://api.github.com/users/kush789/repos","events_url":"https://api.github.com/users/kush789/events{/privacy}","received_events_url":"https://api.github.com/users/kush789/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-01T19:57:56Z","updated_at":"2015-02-02T22:04:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wbhart/flint2/pulls/115","html_url":"https://github.com/wbhart/flint2/pull/115","diff_url":"https://github.com/wbhart/flint2/pull/115.diff","patch_url":"https://github.com/wbhart/flint2/pull/115.patch"},"body":"Hi,\r\nI have added the code for perfect power testing, as mentioned in the todo.txt under fmpz_factor. This function tests whether an integer can be represented in form of a^b. I've updated the documentation accordingly too.\r\n\r\nThere wasn't a lot mentioned in the todo regarding this function. Please do let me know what all has to be changed and added.\r\nThanks,\r\nKushagra"},"comment":{"url":"https://api.github.com/repos/wbhart/flint2/issues/comments/72548059","html_url":"https://github.com/wbhart/flint2/pull/115#issuecomment-72548059","issue_url":"https://api.github.com/repos/wbhart/flint2/issues/115","id":72548059,"user":{"login":"kush789","id":7972943,"avatar_url":"https://avatars.githubusercontent.com/u/7972943?v=3","gravatar_id":"","url":"https://api.github.com/users/kush789","html_url":"https://github.com/kush789","followers_url":"https://api.github.com/users/kush789/followers","following_url":"https://api.github.com/users/kush789/following{/other_user}","gists_url":"https://api.github.com/users/kush789/gists{/gist_id}","starred_url":"https://api.github.com/users/kush789/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kush789/subscriptions","organizations_url":"https://api.github.com/users/kush789/orgs","repos_url":"https://api.github.com/users/kush789/repos","events_url":"https://api.github.com/users/kush789/events{/privacy}","received_events_url":"https://api.github.com/users/kush789/received_events","type":"User","site_admin":false},"created_at":"2015-02-02T22:04:51Z","updated_at":"2015-02-02T22:04:51Z","body":"Hey,\r\n\r\nAs of now the code checks whether n can be expressed as a^b for all possible bases, not just a given base. The base parameter passed is set to the value of the base, if found while checking. Otherwise it is set as -1. The function is not asking for any base from the user. Similarly with the passed argument \"exponent\". But yes, what you are suggesting is much more efficient. I'll try implementing that and get back to you.\r\n\r\nThanks,\r\nKushagra"}}