{"action":"created","issue":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5445","repository_url":"https://api.github.com/repos/scikit-learn/scikit-learn","labels_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5445/labels{/name}","comments_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5445/comments","events_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5445/events","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5445","id":112114532,"number":5445,"title":"[MRG+2] Raise ValueError for metrics.cluster.supervised with too many classes","user":{"login":"tomMoral","id":3321081,"avatar_url":"https://avatars.githubusercontent.com/u/3321081?v=3","gravatar_id":"","url":"https://api.github.com/users/tomMoral","html_url":"https://github.com/tomMoral","followers_url":"https://api.github.com/users/tomMoral/followers","following_url":"https://api.github.com/users/tomMoral/following{/other_user}","gists_url":"https://api.github.com/users/tomMoral/gists{/gist_id}","starred_url":"https://api.github.com/users/tomMoral/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomMoral/subscriptions","organizations_url":"https://api.github.com/users/tomMoral/orgs","repos_url":"https://api.github.com/users/tomMoral/repos","events_url":"https://api.github.com/users/tomMoral/events{/privacy}","received_events_url":"https://api.github.com/users/tomMoral/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2015-10-19T10:09:55Z","updated_at":"2016-09-14T10:18:07Z","closed_at":"2015-10-19T21:31:30Z","pull_request":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/pulls/5445","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5445","diff_url":"https://github.com/scikit-learn/scikit-learn/pull/5445.diff","patch_url":"https://github.com/scikit-learn/scikit-learn/pull/5445.patch"},"body":"Fix for #4976\r\n- Add max_n_classes param to cluster.supervised metric\r\n- Add testing for the Value Error\r\n- Check that n_clusters,n_classes are not too high in contingency matrix"},"comment":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/comments/246968430","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5445#issuecomment-246968430","issue_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5445","id":246968430,"user":{"login":"jnothman","id":78827,"avatar_url":"https://avatars.githubusercontent.com/u/78827?v=3","gravatar_id":"","url":"https://api.github.com/users/jnothman","html_url":"https://github.com/jnothman","followers_url":"https://api.github.com/users/jnothman/followers","following_url":"https://api.github.com/users/jnothman/following{/other_user}","gists_url":"https://api.github.com/users/jnothman/gists{/gist_id}","starred_url":"https://api.github.com/users/jnothman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jnothman/subscriptions","organizations_url":"https://api.github.com/users/jnothman/orgs","repos_url":"https://api.github.com/users/jnothman/repos","events_url":"https://api.github.com/users/jnothman/events{/privacy}","received_events_url":"https://api.github.com/users/jnothman/received_events","type":"User","site_admin":false},"created_at":"2016-09-14T10:18:07Z","updated_at":"2016-09-14T10:18:07Z","body":"The issue here is really a validation issue. If someone puts continuous scores into a clustering metric we can either explicitly catch that and raise an error; otherwise, we should return a result of 0.\r\n\r\nThere are, on the other hand, real clustering problems with a large number of clusters, but any likely clustering is sparse."}}
{"actor":{"display_login":"jnothman"}}