[
https://issues.apache.org/jira/browse/FLINK-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15286461#comment-15286461
]
ASF GitHub Bot commented on FLINK-1979:
---------------------------------------
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/1985#discussion_r63509413
--- Diff:
flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/optimization/GradientDescent.scala
---
@@ -287,11 +287,8 @@ class GradientDescentL2 extends GradientDescent {
regularizationConstant: Double,
learningRate: Double)
: Vector = {
- // add the gradient of the L2 regularization
- BLAS.axpy(regularizationConstant, weightVector, gradient)
- // update the weights according to the learning rate
- BLAS.axpy(-learningRate, gradient, weightVector)
+ L2Regularization.takeStep(weightVector, gradient,
regularizationConstant,learningRate)
--- End diff --
whitespace missing between comma and `learningRate`
> Implement Loss Functions
> ------------------------
>
> Key: FLINK-1979
> URL: https://issues.apache.org/jira/browse/FLINK-1979
> Project: Flink
> Issue Type: Improvement
> Components: Machine Learning Library
> Reporter: Johannes Günther
> Assignee: Johannes Günther
> Priority: Minor
> Labels: ML
>
> For convex optimization problems, optimizer methods like SGD rely on a
> pluggable implementation of a loss function and its first derivative.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
