{"action":"created","issue":{"url":"https://api.github.com/repos/SciTools/iris/issues/1714","labels_url":"https://api.github.com/repos/SciTools/iris/issues/1714/labels{/name}","comments_url":"https://api.github.com/repos/SciTools/iris/issues/1714/comments","events_url":"https://api.github.com/repos/SciTools/iris/issues/1714/events","html_url":"https://github.com/SciTools/iris/pull/1714","id":91258778,"number":1714,"title":"Weights keyword for pearsonr","user":{"login":"rcomer","id":10599679,"avatar_url":"https://avatars.githubusercontent.com/u/10599679?v=3","gravatar_id":"","url":"https://api.github.com/users/rcomer","html_url":"https://github.com/rcomer","followers_url":"https://api.github.com/users/rcomer/followers","following_url":"https://api.github.com/users/rcomer/following{/other_user}","gists_url":"https://api.github.com/users/rcomer/gists{/gist_id}","starred_url":"https://api.github.com/users/rcomer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rcomer/subscriptions","organizations_url":"https://api.github.com/users/rcomer/orgs","repos_url":"https://api.github.com/users/rcomer/repos","events_url":"https://api.github.com/users/rcomer/events{/privacy}","received_events_url":"https://api.github.com/users/rcomer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":21,"created_at":"2015-06-26T13:58:31Z","updated_at":"2015-07-02T09:09:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SciTools/iris/pulls/1714","html_url":"https://github.com/SciTools/iris/pull/1714","diff_url":"https://github.com/SciTools/iris/pull/1714.diff","patch_url":"https://github.com/SciTools/iris/pull/1714.patch"},"body":"Enables specification of `weights` keyword in `iris.analysis.stats.pearsonr`, to calculate a weighted correlation.  I want to use it for spatial correlations.\r\n\r\nIn the unit tests, I changed the \"perfect correlation\" tests from `assertArrayEqual` to `assertArrayAlmostEqual` because I only got 1 to seven decimal places."},"comment":{"url":"https://api.github.com/repos/SciTools/iris/issues/comments/117970415","html_url":"https://github.com/SciTools/iris/pull/1714#issuecomment-117970415","issue_url":"https://api.github.com/repos/SciTools/iris/issues/1714","id":117970415,"user":{"login":"ajdawson","id":390602,"avatar_url":"https://avatars.githubusercontent.com/u/390602?v=3","gravatar_id":"","url":"https://api.github.com/users/ajdawson","html_url":"https://github.com/ajdawson","followers_url":"https://api.github.com/users/ajdawson/followers","following_url":"https://api.github.com/users/ajdawson/following{/other_user}","gists_url":"https://api.github.com/users/ajdawson/gists{/gist_id}","starred_url":"https://api.github.com/users/ajdawson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajdawson/subscriptions","organizations_url":"https://api.github.com/users/ajdawson/orgs","repos_url":"https://api.github.com/users/ajdawson/repos","events_url":"https://api.github.com/users/ajdawson/events{/privacy}","received_events_url":"https://api.github.com/users/ajdawson/received_events","type":"User","site_admin":false},"created_at":"2015-07-02T09:09:10Z","updated_at":"2015-07-02T09:09:10Z","body":"@rcomer - I've made a few more minor comments. The documentation comments may refer to things you didn't write originally, but I think since this is a total rewrite it might be worth addressing them now. I also want to reassure you that although this is taking a long time, the work is of very high quality and I'm very happy that you put in the time to do it all!"}}