Github user gaborhermann commented on a diff in the pull request: https://github.com/apache/flink/pull/2542#discussion_r81112771 --- Diff: docs/dev/libs/ml/als.md --- @@ -99,6 +114,26 @@ The alternating least squares implementation can be controlled by the following </td> </tr> <tr> + <td><strong>ImplicitPrefs</strong></td> + <td> + <p> + Implicit property of the observations, meaning that they do not represent an explicit + preference of the user, just the implicit information how many times the user consumed the + (Default value: <strong>false</strong>) + </p> + </td> + </tr> + <tr> + <td><strong>Alpha</strong></td> + <td> + <p> + Weight of the positive implicit observations. Should be non-negative. + Only relevant when ImplicitPrefs is set to true. + (Default value: <strong>1</strong>) --- End diff -- I guess it was left as default 1 for no reason, I don't have any motivation for it. I changed it to 40 according to the paper. Thanks for the suggestion!
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---