{"action":"created","issue":{"url":"https://api.github.com/repos/PyAbel/PyAbel/issues/85","repository_url":"https://api.github.com/repos/PyAbel/PyAbel","labels_url":"https://api.github.com/repos/PyAbel/PyAbel/issues/85/labels{/name}","comments_url":"https://api.github.com/repos/PyAbel/PyAbel/issues/85/comments","events_url":"https://api.github.com/repos/PyAbel/PyAbel/issues/85/events","html_url":"https://github.com/PyAbel/PyAbel/pull/85","id":133379798,"number":85,"title":"add find_center_by_fit_gaussian","user":{"login":"ldes89150","id":3795568,"avatar_url":"https://avatars.githubusercontent.com/u/3795568?v=3","gravatar_id":"","url":"https://api.github.com/users/ldes89150","html_url":"https://github.com/ldes89150","followers_url":"https://api.github.com/users/ldes89150/followers","following_url":"https://api.github.com/users/ldes89150/following{/other_user}","gists_url":"https://api.github.com/users/ldes89150/gists{/gist_id}","starred_url":"https://api.github.com/users/ldes89150/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldes89150/subscriptions","organizations_url":"https://api.github.com/users/ldes89150/orgs","repos_url":"https://api.github.com/users/ldes89150/repos","events_url":"https://api.github.com/users/ldes89150/events{/privacy}","received_events_url":"https://api.github.com/users/ldes89150/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-02-13T00:50:02Z","updated_at":"2016-02-14T14:35:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/PyAbel/PyAbel/pulls/85","html_url":"https://github.com/PyAbel/PyAbel/pull/85","diff_url":"https://github.com/PyAbel/PyAbel/pull/85.diff","patch_url":"https://github.com/PyAbel/PyAbel/pull/85.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/PyAbel/PyAbel/issues/comments/183901763","html_url":"https://github.com/PyAbel/PyAbel/pull/85#issuecomment-183901763","issue_url":"https://api.github.com/repos/PyAbel/PyAbel/issues/85","id":183901763,"user":{"login":"rth","id":630936,"avatar_url":"https://avatars.githubusercontent.com/u/630936?v=3","gravatar_id":"","url":"https://api.github.com/users/rth","html_url":"https://github.com/rth","followers_url":"https://api.github.com/users/rth/followers","following_url":"https://api.github.com/users/rth/following{/other_user}","gists_url":"https://api.github.com/users/rth/gists{/gist_id}","starred_url":"https://api.github.com/users/rth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rth/subscriptions","organizations_url":"https://api.github.com/users/rth/orgs","repos_url":"https://api.github.com/users/rth/repos","events_url":"https://api.github.com/users/rth/events{/privacy}","received_events_url":"https://api.github.com/users/rth/received_events","type":"User","site_admin":false},"created_at":"2016-02-14T14:35:46Z","updated_at":"2016-02-14T14:35:46Z","body":"Nice work, @ldes89150 !\r\n\r\nJust a minor comment, `round` is a builtin Python function, that already does what you want (and returns an int). So you don't actually need `scipy.special.round`  (which is an alias  for `numpy.round` BTW). Therefore I would suggest to just remove the `scipy.special` import, using the builtin function (i.e. replace `int(scipy.special.round` by `round`), and possibly avoid redefining  builtin's  (so use e.g. `round_func = .. ` instead of `round = .. `). \r\n\r\nAlso I don't think that the  `_` function name follows usual Python naming conventions, maybe it could be replaced by `_f` to be more explicit?  Feel free to edit the Gaussian definition in  `analytical.py` to add  the missing constant parameter (with a default to 0, so it does not break other tests) if you want, or we can also keep both versions for now.\r\n  \r\nOtherwise this looks good, thanks a lot for contributing! "}}