[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-05-30 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r884843120 ## flink-ml-lib/src/main/java/org/apache/flink/ml/classification/logisticregression/OnlineLogisticRegression.java: ## @@ -0,0 +1,434 @@ +/* + * Licensed to the Apache

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-05-30 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r884764718 ## flink-ml-lib/src/main/java/org/apache/flink/ml/classification/logisticregression/OnlineLogisticRegressionParams.java: ## @@ -0,0 +1,66 @@ +/* + * Licensed to the

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-05-30 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r884756711 ## flink-ml-lib/src/main/java/org/apache/flink/ml/classification/logisticregression/OnlineLogisticRegression.java: ## @@ -0,0 +1,434 @@ +/* + * Licensed to the Apache

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-05-30 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r884756711 ## flink-ml-lib/src/main/java/org/apache/flink/ml/classification/logisticregression/OnlineLogisticRegression.java: ## @@ -0,0 +1,434 @@ +/* + * Licensed to the Apache

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-05-27 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r883628874 ## flink-ml-core/src/main/java/org/apache/flink/ml/common/datastream/DataStreamUtils.java: ## @@ -182,4 +193,63 @@ public void snapshotState(StateSnapshotContext

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-04-28 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r860633423 ## flink-ml-lib/src/main/java/org/apache/flink/ml/common/param/HasL2.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-04-28 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r860550025 ## flink-ml-lib/src/main/java/org/apache/flink/ml/classification/logisticregression/LogisticRegressionModel.java: ## @@ -162,9 +164,17 @@ public Row map(Row dataPoint) {

[GitHub] [flink-ml] lindong28 commented on a diff in pull request #83: [FLINK-27170] Add Transformer and Estimator for OnlineLogisticRegression

2022-04-28 Thread GitBox
lindong28 commented on code in PR #83: URL: https://github.com/apache/flink-ml/pull/83#discussion_r860545252 ## flink-ml-lib/src/main/java/org/apache/flink/ml/common/param/HasL2.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or