{"action":"created","issue":{"url":"https://api.github.com/repos/apache/flink/issues/2542","repository_url":"https://api.github.com/repos/apache/flink","labels_url":"https://api.github.com/repos/apache/flink/issues/2542/labels{/name}","comments_url":"https://api.github.com/repos/apache/flink/issues/2542/comments","events_url":"https://api.github.com/repos/apache/flink/issues/2542/events","html_url":"https://github.com/apache/flink/pull/2542","id":178849808,"number":2542,"title":"[FLINK-4613] [ml] Extend ALS to handle implicit feedback datasets","user":{"login":"gaborhermann","id":6912263,"avatar_url":"https://avatars.githubusercontent.com/u/6912263?v=3","gravatar_id":"","url":"https://api.github.com/users/gaborhermann","html_url":"https://github.com/gaborhermann","followers_url":"https://api.github.com/users/gaborhermann/followers","following_url":"https://api.github.com/users/gaborhermann/following{/other_user}","gists_url":"https://api.github.com/users/gaborhermann/gists{/gist_id}","starred_url":"https://api.github.com/users/gaborhermann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gaborhermann/subscriptions","organizations_url":"https://api.github.com/users/gaborhermann/orgs","repos_url":"https://api.github.com/users/gaborhermann/repos","events_url":"https://api.github.com/users/gaborhermann/events{/privacy}","received_events_url":"https://api.github.com/users/gaborhermann/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2016-09-23T11:11:20Z","updated_at":"2016-09-30T06:57:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/flink/pulls/2542","html_url":"https://github.com/apache/flink/pull/2542","diff_url":"https://github.com/apache/flink/pull/2542.diff","patch_url":"https://github.com/apache/flink/pull/2542.patch"},"body":"This extension of the ALS algorithm changes some parts of the code if `implicitPrefs` flag is set to true. Mainly the local parts parts are changed: the `Xt * X` computation takes into consideration the confidence, thus computing `Xt * (C - I) * X` instead (see the paper by Hu et al. for details). The `Xt * X` matrix is precomputed and broadcasted, and that is the only thing that affects distributed execution.\r\n\r\nNote, that we use a temporary directory in the test, because there would not be enough memory segments to perform a hash join for prediction. I assume that memory segments are not freed up after the training if no temporary directory is set, but I did not investigate the issue as using a tempdir is a simple workaround."},"comment":{"url":"https://api.github.com/repos/apache/flink/issues/comments/250673531","html_url":"https://github.com/apache/flink/pull/2542#issuecomment-250673531","issue_url":"https://api.github.com/repos/apache/flink/issues/2542","id":250673531,"user":{"login":"mbalassi","id":5990983,"avatar_url":"https://avatars.githubusercontent.com/u/5990983?v=3","gravatar_id":"","url":"https://api.github.com/users/mbalassi","html_url":"https://github.com/mbalassi","followers_url":"https://api.github.com/users/mbalassi/followers","following_url":"https://api.github.com/users/mbalassi/following{/other_user}","gists_url":"https://api.github.com/users/mbalassi/gists{/gist_id}","starred_url":"https://api.github.com/users/mbalassi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbalassi/subscriptions","organizations_url":"https://api.github.com/users/mbalassi/orgs","repos_url":"https://api.github.com/users/mbalassi/repos","events_url":"https://api.github.com/users/mbalassi/events{/privacy}","received_events_url":"https://api.github.com/users/mbalassi/received_events","type":"User","site_admin":false},"created_at":"2016-09-30T06:57:24Z","updated_at":"2016-09-30T06:57:24Z","body":"@gaborhermann @thvasilo I would definitely like to see a test on a larger dataset, that is actually what I was asking for when I mentioned \"benchmark\", maybe I was not clear then."}}
{"actor":{"display_login":"mbalassi"}}