{"action":"created","issue":{"url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/138","repository_url":"https://api.github.com/repos/TUW-GEO/pytesmo","labels_url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/138/labels{/name}","comments_url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/138/comments","events_url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/138/events","html_url":"https://github.com/TUW-GEO/pytesmo/issues/138","id":337036676,"node_id":"MDU6SXNzdWUzMzcwMzY2NzY=","number":138,"title":"bias calculation","user":{"login":"wpreimes","id":24410143,"node_id":"MDQ6VXNlcjI0NDEwMTQz","avatar_url":"https://avatars2.githubusercontent.com/u/24410143?v=4","gravatar_id":"","url":"https://api.github.com/users/wpreimes","html_url":"https://github.com/wpreimes","followers_url":"https://api.github.com/users/wpreimes/followers","following_url":"https://api.github.com/users/wpreimes/following{/other_user}","gists_url":"https://api.github.com/users/wpreimes/gists{/gist_id}","starred_url":"https://api.github.com/users/wpreimes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wpreimes/subscriptions","organizations_url":"https://api.github.com/users/wpreimes/orgs","repos_url":"https://api.github.com/users/wpreimes/repos","events_url":"https://api.github.com/users/wpreimes/events{/privacy}","received_events_url":"https://api.github.com/users/wpreimes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2018-06-29T15:12:18Z","updated_at":"2018-07-03T13:48:07Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"Hey! Bias is currently np.mean(observations) - np.mean(predictions), but should be the other way around, right? @iteubner also mentioned this some time ago."},"comment":{"url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/comments/402163596","html_url":"https://github.com/TUW-GEO/pytesmo/issues/138#issuecomment-402163596","issue_url":"https://api.github.com/repos/TUW-GEO/pytesmo/issues/138","id":402163596,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMjE2MzU5Ng==","user":{"login":"cpaulik","id":380927,"node_id":"MDQ6VXNlcjM4MDkyNw==","avatar_url":"https://avatars0.githubusercontent.com/u/380927?v=4","gravatar_id":"","url":"https://api.github.com/users/cpaulik","html_url":"https://github.com/cpaulik","followers_url":"https://api.github.com/users/cpaulik/followers","following_url":"https://api.github.com/users/cpaulik/following{/other_user}","gists_url":"https://api.github.com/users/cpaulik/gists{/gist_id}","starred_url":"https://api.github.com/users/cpaulik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cpaulik/subscriptions","organizations_url":"https://api.github.com/users/cpaulik/orgs","repos_url":"https://api.github.com/users/cpaulik/repos","events_url":"https://api.github.com/users/cpaulik/events{/privacy}","received_events_url":"https://api.github.com/users/cpaulik/received_events","type":"User","site_admin":false},"created_at":"2018-07-03T13:48:07Z","updated_at":"2018-07-03T13:48:07Z","author_association":"MEMBER","body":"I would not like to switch the argument order since this would lead to all existing code suddenly producing different results.\r\n\r\nAs far as I've seen by quickly going through the code bias is the only metric we have where the argument order matters.\r\n\r\nAbout the documentation. I've actually liked the way it was before `o` and `p` where introduced where we just had `x` and `y` and the calling function has to decide in which direction the bias is interesting based on the application. E.g. the validation framework [call the bias function with `bias(ref, other)`](https://github.com/TUW-GEO/pytesmo/blob/master/pytesmo/validation_framework/metric_calculators.py#L116). "}}
{"actor":{"display_login":"cpaulik"}}