{"action":"created","issue":{"url":"https://api.github.com/repos/brettc/partitionfinder/issues/44","labels_url":"https://api.github.com/repos/brettc/partitionfinder/issues/44/labels{/name}","comments_url":"https://api.github.com/repos/brettc/partitionfinder/issues/44/comments","events_url":"https://api.github.com/repos/brettc/partitionfinder/issues/44/events","html_url":"https://github.com/brettc/partitionfinder/issues/44","id":88721621,"number":44,"title":"We still call fast_TIGER instead of the new cython tiger implementation for rate calculation","user":{"login":"pbfrandsen","id":1823345,"avatar_url":"https://avatars.githubusercontent.com/u/1823345?v=3","gravatar_id":"","url":"https://api.github.com/users/pbfrandsen","html_url":"https://github.com/pbfrandsen","followers_url":"https://api.github.com/users/pbfrandsen/followers","following_url":"https://api.github.com/users/pbfrandsen/following{/other_user}","gists_url":"https://api.github.com/users/pbfrandsen/gists{/gist_id}","starred_url":"https://api.github.com/users/pbfrandsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pbfrandsen/subscriptions","organizations_url":"https://api.github.com/users/pbfrandsen/orgs","repos_url":"https://api.github.com/users/pbfrandsen/repos","events_url":"https://api.github.com/users/pbfrandsen/events{/privacy}","received_events_url":"https://api.github.com/users/pbfrandsen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/brettc/partitionfinder/milestones/2","html_url":"https://github.com/brettc/partitionfinder/milestones/PF2.0%20Beta","labels_url":"https://api.github.com/repos/brettc/partitionfinder/milestones/2/labels","id":1226678,"number":2,"title":"PF2.0 Beta","description":"A first cut of the new beta.","creator":{"login":"brettc","id":457283,"avatar_url":"https://avatars.githubusercontent.com/u/457283?v=3","gravatar_id":"","url":"https://api.github.com/users/brettc","html_url":"https://github.com/brettc","followers_url":"https://api.github.com/users/brettc/followers","following_url":"https://api.github.com/users/brettc/following{/other_user}","gists_url":"https://api.github.com/users/brettc/gists{/gist_id}","starred_url":"https://api.github.com/users/brettc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brettc/subscriptions","organizations_url":"https://api.github.com/users/brettc/orgs","repos_url":"https://api.github.com/users/brettc/repos","events_url":"https://api.github.com/users/brettc/events{/privacy}","received_events_url":"https://api.github.com/users/brettc/received_events","type":"User","site_admin":false},"open_issues":6,"closed_issues":2,"state":"open","created_at":"2015-07-27T01:31:29Z","updated_at":"2015-08-03T23:14:02Z","due_on":"2015-08-30T12:00:00Z","closed_at":null},"comments":8,"created_at":"2015-06-16T13:01:08Z","updated_at":"2015-08-04T07:28:13Z","closed_at":null,"body":"We are still calling fast_TIGER within kmeans.py gen_per_site_stats(). I thought that we made the change to the cython tiger rates during the workshop, but I can't find that function anywhere."},"comment":{"url":"https://api.github.com/repos/brettc/partitionfinder/issues/comments/127506880","html_url":"https://github.com/brettc/partitionfinder/issues/44#issuecomment-127506880","issue_url":"https://api.github.com/repos/brettc/partitionfinder/issues/44","id":127506880,"user":{"login":"roblanf","id":895251,"avatar_url":"https://avatars.githubusercontent.com/u/895251?v=3","gravatar_id":"","url":"https://api.github.com/users/roblanf","html_url":"https://github.com/roblanf","followers_url":"https://api.github.com/users/roblanf/followers","following_url":"https://api.github.com/users/roblanf/following{/other_user}","gists_url":"https://api.github.com/users/roblanf/gists{/gist_id}","starred_url":"https://api.github.com/users/roblanf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/roblanf/subscriptions","organizations_url":"https://api.github.com/users/roblanf/orgs","repos_url":"https://api.github.com/users/roblanf/repos","events_url":"https://api.github.com/users/roblanf/events{/privacy}","received_events_url":"https://api.github.com/users/roblanf/received_events","type":"User","site_admin":false},"created_at":"2015-08-04T07:28:13Z","updated_at":"2015-08-04T07:28:13Z","body":"so, here are the results so far. Entropy usually does better (often by a fair bit), and occasionally does a little worse than tiger. \r\n\r\nUnless the last few datasets throw up some drastic surprises, my vote is to move kmeans to work only with entropy.\r\n\r\nHere are the AICc scores on the first few datasets\r\n\r\n![rplot](https://cloud.githubusercontent.com/assets/895251/9055350/b414ed46-3acd-11e5-8805-462e06a17546.jpg)\r\n\r\nOne interesting thing that this figure doesn't show is that entropy kmeans is doing all of this while selecting drastically fewer subsets. This seems like a very good thing, given the tendency in simulation for tiger kmeans to oversplit. In fact, in every case, entropy kmeans gets FEWER subsets than the greedy algorithm. \r\n\r\n![rplot](https://cloud.githubusercontent.com/assets/895251/9055392/1f33b5b2-3ace-11e5-8348-5aa57f7adf85.jpg)\r\n\r\n\r\n\r\n"}}