{"action":"created","issue":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5295","repository_url":"https://api.github.com/repos/scikit-learn/scikit-learn","labels_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5295/labels{/name}","comments_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5295/comments","events_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5295/events","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5295","id":107751846,"number":5295,"title":"[MRG+1] Add multiplicative-update solver in NMF, with all beta-divergence","user":{"login":"TomDLT","id":11065596,"avatar_url":"https://avatars.githubusercontent.com/u/11065596?v=3","gravatar_id":"","url":"https://api.github.com/users/TomDLT","html_url":"https://github.com/TomDLT","followers_url":"https://api.github.com/users/TomDLT/followers","following_url":"https://api.github.com/users/TomDLT/following{/other_user}","gists_url":"https://api.github.com/users/TomDLT/gists{/gist_id}","starred_url":"https://api.github.com/users/TomDLT/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomDLT/subscriptions","organizations_url":"https://api.github.com/users/TomDLT/orgs","repos_url":"https://api.github.com/users/TomDLT/repos","events_url":"https://api.github.com/users/TomDLT/events{/privacy}","received_events_url":"https://api.github.com/users/TomDLT/received_events","type":"User","site_admin":false},"labels":[{"id":297514516,"url":"https://api.github.com/repos/scikit-learn/scikit-learn/labels/Needs%20Review","name":"Needs Review","color":"f7c6c7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":52,"created_at":"2015-09-22T16:32:02Z","updated_at":"2016-12-12T10:27:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/pulls/5295","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5295","diff_url":"https://github.com/scikit-learn/scikit-learn/pull/5295.diff","patch_url":"https://github.com/scikit-learn/scikit-learn/pull/5295.patch"},"body":"This PR is a second part of what have been discussed in #4811. (first part, #4852, merged)\nIt includes :\n- a Multiplicative-Update solver in NMF. This solver is generally slower than the Coordinate-Descent solver, but it handles all beta-divergences (including Frobenius, generalized Kullback_Leibler and Itakura-Saito). \n- a plot to visualize the beta-divergence for several values of beta.\n- benchmarks [below](https://github.com/scikit-learn/scikit-learn/pull/5295#issuecomment-174002806)\n\n[Link to the rendered doc](https://5075-843222-gh.circle-artifacts.com/0//home/ubuntu/scikit-learn/doc/_build/html/stable/modules/decomposition.html#nmf-with-a-beta-divergence)\n\n---\n\nplot_beta_divergence.py\n![beta_div](https://cloud.githubusercontent.com/assets/11065596/10021621/232917e2-6149-11e5-84f6-8e9b09578216.png)\n\n<!-- Reviewable:start -->\n\n---\n\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/scikit-learn/scikit-learn/5295)\n\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/comments/266393130","html_url":"https://github.com/scikit-learn/scikit-learn/pull/5295#issuecomment-266393130","issue_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/5295","id":266393130,"user":{"login":"ogrisel","id":89061,"avatar_url":"https://avatars.githubusercontent.com/u/89061?v=3","gravatar_id":"","url":"https://api.github.com/users/ogrisel","html_url":"https://github.com/ogrisel","followers_url":"https://api.github.com/users/ogrisel/followers","following_url":"https://api.github.com/users/ogrisel/following{/other_user}","gists_url":"https://api.github.com/users/ogrisel/gists{/gist_id}","starred_url":"https://api.github.com/users/ogrisel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ogrisel/subscriptions","organizations_url":"https://api.github.com/users/ogrisel/orgs","repos_url":"https://api.github.com/users/ogrisel/repos","events_url":"https://api.github.com/users/ogrisel/events{/privacy}","received_events_url":"https://api.github.com/users/ogrisel/received_events","type":"User","site_admin":false},"created_at":"2016-12-12T10:27:00Z","updated_at":"2016-12-12T10:27:00Z","body":"@TomDLT now that #7927 has been merged, this needs a rebase."}}
{"actor":{"display_login":"ogrisel"}}