{"action":"created","issue":{"url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/1","labels_url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/1/comments","events_url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/1/events","html_url":"https://github.com/PhilippsCat/MarkovChains/issues/1","id":78999760,"number":1,"title":"Hello","user":{"login":"nwulkow","id":12544507,"avatar_url":"https://avatars.githubusercontent.com/u/12544507?v=3","gravatar_id":"","url":"https://api.github.com/users/nwulkow","html_url":"https://github.com/nwulkow","followers_url":"https://api.github.com/users/nwulkow/followers","following_url":"https://api.github.com/users/nwulkow/following{/other_user}","gists_url":"https://api.github.com/users/nwulkow/gists{/gist_id}","starred_url":"https://api.github.com/users/nwulkow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nwulkow/subscriptions","organizations_url":"https://api.github.com/users/nwulkow/orgs","repos_url":"https://api.github.com/users/nwulkow/repos","events_url":"https://api.github.com/users/nwulkow/events{/privacy}","received_events_url":"https://api.github.com/users/nwulkow/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-05-21T13:39:59Z","updated_at":"2015-06-28T17:49:15Z","closed_at":"2015-06-28T17:49:15Z","body":"Hi, this is were we can communicate"},"comment":{"url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/comments/116303587","html_url":"https://github.com/PhilippsCat/MarkovChains/issues/1#issuecomment-116303587","issue_url":"https://api.github.com/repos/PhilippsCat/MarkovChains/issues/1","id":116303587,"user":{"login":"c1887","id":7395967,"avatar_url":"https://avatars.githubusercontent.com/u/7395967?v=3","gravatar_id":"","url":"https://api.github.com/users/c1887","html_url":"https://github.com/c1887","followers_url":"https://api.github.com/users/c1887/followers","following_url":"https://api.github.com/users/c1887/following{/other_user}","gists_url":"https://api.github.com/users/c1887/gists{/gist_id}","starred_url":"https://api.github.com/users/c1887/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/c1887/subscriptions","organizations_url":"https://api.github.com/users/c1887/orgs","repos_url":"https://api.github.com/users/c1887/repos","events_url":"https://api.github.com/users/c1887/events{/privacy}","received_events_url":"https://api.github.com/users/c1887/received_events","type":"User","site_admin":false},"created_at":"2015-06-28T17:49:15Z","updated_at":"2015-06-28T17:49:15Z","body":"@nwulkow: Thank you for implementing the regspace method. Anyway, there might be room for improvement.\r\nFirst of all, please check your indentings. Your method should be on the same level as the kmeans method. Right now, it is an sub method of it. Maybe it would be good to test your code in the python command line (not only in an ipython notebook) to avoid these little things.\r\n\r\nAfter that, I still have problems testing your code. It seems to depend highly on the right dimensions of the input data, as I get (on the first glance) reasonable results but that are always the same even for different dmin. \r\nLooking at your code, I also don't understand why you check for __<__d_min. I mean don't we create a new cluster if it's too far _away_ from the others?\r\n\r\nAnd in the end, one thing that would be nice is if we could also get a discrete trajectory from it like in the kmeans method, I mean something with has only integer states in it (e.g. [0,4,2,1,3,3,4]...), so we can put it directly into the estimation methods."}}