Github user gaborhermann commented on a diff in the pull request:
https://github.com/apache/flink/pull/2542#discussion_r87587473
--- Diff:
flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/recommendation/ALS.scala
---
@@ -273,6 +308,14 @@ object ALS {
val defaultValue: Option[Int] = Some(10)
}
+ case object ImplicitPrefs extends Parameter[Boolean] {
--- End diff --
I suggest to leave it at 10 and do not give any advise to use highest
possible number. It should work out-of-the-box, and to get good results, the
user must fit the parameters to the use-case (such as the lambda coefficient of
the regularization). Btw. Spark uses 10 by default.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
