{"action":"created","issue":{"url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/3","labels_url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/3/comments","events_url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/3/events","html_url":"https://github.com/fiji/Auto_Threshold/pull/3","id":106708308,"number":3,"title":"Fixed normalization for Phansalkar's method","user":{"login":"stelfrich","id":333358,"avatar_url":"https://avatars.githubusercontent.com/u/333358?v=3","gravatar_id":"","url":"https://api.github.com/users/stelfrich","html_url":"https://github.com/stelfrich","followers_url":"https://api.github.com/users/stelfrich/followers","following_url":"https://api.github.com/users/stelfrich/following{/other_user}","gists_url":"https://api.github.com/users/stelfrich/gists{/gist_id}","starred_url":"https://api.github.com/users/stelfrich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stelfrich/subscriptions","organizations_url":"https://api.github.com/users/stelfrich/orgs","repos_url":"https://api.github.com/users/stelfrich/repos","events_url":"https://api.github.com/users/stelfrich/events{/privacy}","received_events_url":"https://api.github.com/users/stelfrich/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-09-16T06:28:26Z","updated_at":"2015-09-16T15:47:50Z","closed_at":"2015-09-16T15:47:19Z","pull_request":{"url":"https://api.github.com/repos/fiji/Auto_Threshold/pulls/3","html_url":"https://github.com/fiji/Auto_Threshold/pull/3","diff_url":"https://github.com/fiji/Auto_Threshold/pull/3.diff","patch_url":"https://github.com/fiji/Auto_Threshold/pull/3.patch"},"body":"In the current implementation of Phansalkar's method, the `ContrastEnhancer` doesn't stretch the histogram because normalization is disabled by default. Hence, the following multiplications by 1/255 (which are done to normalize the values to the [0, 1] range) will actually map the values to [0, maxValue/255].\r\n\r\nWhile the computations are not completely off, the default value of the maximum standard deviation of 0.5 doesn't really make sense if the normalization is incorrect.. "},"comment":{"url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/comments/140782998","html_url":"https://github.com/fiji/Auto_Threshold/pull/3#issuecomment-140782998","issue_url":"https://api.github.com/repos/fiji/Auto_Threshold/issues/3","id":140782998,"user":{"login":"landinig","id":8421792,"avatar_url":"https://avatars.githubusercontent.com/u/8421792?v=3","gravatar_id":"","url":"https://api.github.com/users/landinig","html_url":"https://github.com/landinig","followers_url":"https://api.github.com/users/landinig/followers","following_url":"https://api.github.com/users/landinig/following{/other_user}","gists_url":"https://api.github.com/users/landinig/gists{/gist_id}","starred_url":"https://api.github.com/users/landinig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/landinig/subscriptions","organizations_url":"https://api.github.com/users/landinig/orgs","repos_url":"https://api.github.com/users/landinig/repos","events_url":"https://api.github.com/users/landinig/events{/privacy}","received_events_url":"https://api.github.com/users/landinig/received_events","type":"User","site_admin":false},"created_at":"2015-09-16T15:47:50Z","updated_at":"2015-09-16T15:47:50Z","body":"just did!"}}