Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/35
@maropu test is failing for v2.1.
```
Saving to
outputFile=/home/travis/build/apache/incubator-hivemall/spark/spark-common/target/scalastyle-output.xml
Processed 3 file(s
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/35
BTW, you can close https://github.com/apache/incubator-hivemall/pull/23 as
well.
`Close #35, #23: `
---
If your project is set up for it, you can reply to this email and have
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/37
Benchmark on more large dataset would be helpful (not to be included the
unit test though).
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/36
Fixed in
https://github.com/apache/incubator-hivemall/commit/fdb4dd86965f8b79f6dc257a783e683c93fadc43
It's required for gitbook menu
http://hivemall.incubator.apache.org/userguide
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/41
@amaya382 @Lewuathe Could you confirm that the updated `bin/spark-shell`
properly works if your spare time?
---
If your project is set up for it, you can reply to this email and have your
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/42
BTW, documentation about sst/changefinder on dataframe would be helpful.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/42
@maropu LGTM. Could you merge and close this one?
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/41
@maropu BTW, better to update `incubator-hivemall-site` once this PR is
merged.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/37
@maropu LGTM. Please merge this PR and close the ticket as fixed.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/22
@wangyum @maropu Thanks. Merged with some modifications. Configuration for
spark-1.6 should also be changed.
---
If your project is set up for it, you can reply to this email and have your
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/24
LGTM. Could you update ASF master as follows:
```
git checkout master
git checkout -b CheckCompiler
git pull https://github.com/maropu/incubator-hivemall.git
hotfix
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/31
LGTM. Please merge and close this PR.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/30
@maropu Could you add [SPARK] after [HIVEMALL-37] in the title?
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/33
output `top1Df` should be explained in the description.
BTW, I personally prefer `top_k_join` instead of `join_top_k`.
---
If your project is set up for it, you can reply
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/25
@maropu LGTM. Please merge this PR.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/26
@maropu LGTM. Please merge this PR and close JIRA issue as fixed.
---
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
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97726633
--- Diff: core/src/main/java/hivemall/optimizer/Optimizer.java ---
@@ -0,0 +1,246 @@
+/*
+ * Hivemall: Hive scalable Machine Learning Library
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97936571
--- Diff: core/src/main/java/hivemall/optimizer/Optimizer.java ---
@@ -0,0 +1,246 @@
+/*
+ * Hivemall: Hive scalable Machine Learning Library
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97724411
--- Diff: core/src/main/java/hivemall/classifier/GeneralClassifierUDTF.java
---
@@ -0,0 +1,122 @@
+/*
+ * Licensed to the Apache Software
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97716028
--- Diff: core/src/main/java/hivemall/model/NewDenseModel.java ---
@@ -0,0 +1,293 @@
+/*
+ * Hivemall: Hive scalable Machine Learning Library
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97722648
--- Diff: core/src/main/java/hivemall/optimizer/EtaEstimator.java ---
@@ -157,4 +158,34 @@ public static EtaEstimator get(@Nullable CommandLine
cl
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97715666
--- Diff: core/src/main/java/hivemall/classifier/GeneralClassifierUDTF.java
---
@@ -0,0 +1,122 @@
+/*
+ * Hivemall: Hive scalable Machine
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97716142
--- Diff: core/src/main/java/hivemall/model/NewDenseModel.java ---
@@ -0,0 +1,293 @@
+/*
+ * Hivemall: Hive scalable Machine Learning Library
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97937226
--- Diff: core/src/main/java/hivemall/optimizer/Regularization.java ---
@@ -0,0 +1,99 @@
+/*
+ * Hivemall: Hive scalable Machine Learning
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/14#discussion_r97715621
--- Diff:
core/src/main/java/hivemall/model/NewSpaceEfficientDenseModel.java ---
@@ -0,0 +1,317 @@
+/*
+ * Hivemall: Hive scalable Machine
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/14
LossFunction should be selectable, not fixed.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/27
@maropu LGTM. Please merge this PR and close the JIRA ticket.
BTW, `HiveUdfWithFeatureSuite` causes OOM again.
---
If your project is set up for it, you can reply to this email
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/28
@maropu LGTM. Please merge it.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/29
@maropu LGTM. Please merge and close this PR.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/46
Oops.. the latter one is expected.
```sql
select rescale(4.2,1.0,3.0),rescale(-0.3, 1.0, 3.0);
> 3.0 1.0
select rescale(4.2,1.0,3.0),rescale(-0.3, 1.0,
GitHub user myui opened a pull request:
https://github.com/apache/incubator-hivemall/pull/46
[HIVEMALL-72] Fix rescale UDF behavior to return range [0.0,1.0]
## What changes were proposed in this pull request?
Fix rescale UDF behavior to return range `[0.0,1.0
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/45
@wangyum Thanks. Merged with some modifications.
```sql
select rescale(v,min,max) from (
select cast(1.2 as float) as v, 1.0 as min, 10.0 as max
union all
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/46
@wangyum Are you agree with the following new behavior of
[rescale(value,min,max)](https://en.wikipedia.org/wiki/Feature_scaling#Rescaling)?
```
-- new
select rescale
GitHub user myui opened a pull request:
https://github.com/apache/incubator-hivemall/pull/47
[HIVEMALL-73] Reduce memory usages of each_top_k
## What changes were proposed in this pull request?
Reduce memory usage of each_top_k
## What type of PR
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/50
@takuti LGTM. Thanks!
---
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
GitHub user myui opened a pull request:
https://github.com/apache/incubator-hivemall/pull/51
[WIP][HIVEMALL-75] Support Sparse Vector Format as the input of RandomForest
## What changes were proposed in this pull request?
Supported sparse vector as the input of RandomForest
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/49
@maropu Thanks. LGTM. Merged w/ modifications.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/48
Merged because this PR is a blocker for
https://issues.apache.org/jira/browse/HIVEMALL-75
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/49
Update the project site
http://hivemall.incubator.apache.org/userguide/spark/binaryclass/
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/49
@maropu you need to update `SUMMARY.md` as well for `bin/build_site.sh`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user myui closed the pull request at:
https://github.com/apache/incubator-hivemall/pull/47
---
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
GitHub user myui opened a pull request:
https://github.com/apache/incubator-hivemall/pull/48
[WIP][HIVEMALL-77]
## What changes were proposed in this pull request?
Support CSRMatrix and DenseMatrix in hivemall.matrix package.
## What type of PR
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/41
@maropu Could you update `incubator-hivemall-site`? This feature should be
documented.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/44
@maropu You can use Reflections.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/41
@amaya382 Have you confirmed?
@maropu please merge this PR if he say fine.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/44
@wangyum Merged. Thank you for your contribution!
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/47
Oops. This PR contains a bug. Reverse order to `_queue.poll()` is required
for the output.
```
for (int i = queueSize - 1; i >= 0; i--) {
TupleWithKey tuple = tuple
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/47
hmm... hard to cope w/ this issue. Any good idea? @maropu
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/47
@maropu I found that `each_top_k` behavior on Spark is little bit
difference one from Hive for the ranking scheme in
https://github.com/apache/incubator-hivemall/blob/master/core/src
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/20
@wangyum Thank you for the contribution.
@maropu Could you review this one?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103388346
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -49,35 +50,251 @@
@SuppressWarnings("deprecation")
@D
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103387903
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -49,35 +50,251 @@
@SuppressWarnings("deprecation")
@D
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103386937
--- Diff: core/src/test/java/hivemall/evaluation/AUCUDAFTest.java ---
@@ -0,0 +1,218 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103402140
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -49,35 +51,264 @@
@SuppressWarnings("deprecation")
@D
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103403546
--- Diff: docs/gitbook/eval/auc.md ---
@@ -0,0 +1,102 @@
+
+
+
+
+# Area Under the ROC Curve
+
+[ROC
curve](https
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/52#discussion_r103402767
--- Diff: docs/gitbook/eval/auc.md ---
@@ -0,0 +1,102 @@
+
+
+
+
+# Area Under the ROC Curve
+
+[ROC
curve](https
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/56
LGTM ð
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/53
LGTM ð Thank you for the contribution!
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/42
@maropu sure. Merged! Thanks.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/54
@maropu Merged. Thanks!
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/65
Thanks. Merged into master.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/64
LGTM. Merged. Thanks!
---
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
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109558479
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,497 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109558215
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,497 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109558520
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAUDTF.java ---
@@ -0,0 +1,112 @@
+package hivemall.lda;
+
+import
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109558404
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,497 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109578438
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109577680
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109580335
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109580061
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109578284
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109578413
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109577460
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r109577240
--- Diff: core/src/main/java/hivemall/lda/OnlineLDAModel.java ---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@takuti If you are going to create predict UDAF, better to rename to `lda`
to `train_lda`.
Prediction expected to have feature vector as input and a predicted topic
(& a poste
GitHub user myui opened a pull request:
https://github.com/apache/incubator-hivemall/pull/70
[WIP][HIVEMALL-75-2] Update RandomForest usage
## What changes were proposed in this pull request?
Add `tree_export` UDF and update RandomForest tutorial
## What type of PR
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/68
Thanks! I'll look into it.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/63
@takuti terminatePartial/merge OIs are invalid ones.
---
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
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/63#discussion_r110554001
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -188,13 +234,19 @@ public void iterate(AggregationBuffer agg, Object
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/63#discussion_r110553905
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -35,7 +39,9 @@
import
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/63#discussion_r110553873
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -204,21 +256,53 @@ public void merge(AggregationBuffer agg, Object
partial
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/63#discussion_r110553945
--- Diff: core/src/main/java/hivemall/evaluation/AUCUDAF.java ---
@@ -204,21 +256,53 @@ public void merge(AggregationBuffer agg, Object
partial
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/51
Merged. Documentation and model conversion pull requests follows in
[HIVEMALL-75-2]
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/67
@maropu Thanks! Merged.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@takuti Could you change package from `hivemall.clustering` to
`hivemall.topicmodel`?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/63
@takuti LGTM. Merged to small modifications. Thanks!
---
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
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111727992
--- Diff: core/src/main/java/hivemall/topicmodel/OnlineLDAModel.java ---
@@ -350,66 +348,80 @@ private float computeApproxBoundForMiniBatch
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111728040
--- Diff: core/src/main/java/hivemall/topicmodel/OnlineLDAModel.java ---
@@ -350,66 +348,80 @@ private float computeApproxBoundForMiniBatch
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111728528
--- Diff: core/src/main/java/hivemall/topicmodel/OnlineLDAModel.java ---
@@ -350,66 +348,80 @@ private float computeApproxBoundForMiniBatch
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111728398
--- Diff: core/src/main/java/hivemall/topicmodel/OnlineLDAModel.java ---
@@ -350,66 +348,80 @@ private float computeApproxBoundForMiniBatch
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/71#discussion_r112696086
--- Diff: core/src/main/java/hivemall/topicmodel/IncrementalPLSAModel.java
---
@@ -0,0 +1,300 @@
+/*
+ * Licensed to the Apache Software
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/71#discussion_r112697171
--- Diff: core/src/main/java/hivemall/topicmodel/IncrementalPLSAModel.java
---
@@ -0,0 +1,300 @@
+/*
+ * Licensed to the Apache Software
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/68#discussion_r111326492
--- Diff: docs/gitbook/docker/getting_started.md ---
@@ -0,0 +1,42 @@
+
+
+# Getting Started
+
+## How to run
+1. Install
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/68#discussion_r111327964
--- Diff: .dockerignore ---
@@ -0,0 +1,6 @@
+.dockerignore
+Dockerfile
+docker-compose.yml
--- End diff --
Top-level files
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111548963
--- Diff: core/src/main/java/hivemall/topicmodel/LDAUDTF.java ---
@@ -0,0 +1,553 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/66#discussion_r111549092
--- Diff: core/src/main/java/hivemall/topicmodel/LDAUDTF.java ---
@@ -0,0 +1,553 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@takuti Merged w/ some refactoring. Great work! Thanks.
---
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
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/68
@amaya382 Thanks. I'll try this PR in this week.
---
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
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/71#discussion_r112865163
--- Diff: core/src/main/java/hivemall/utils/math/MathUtils.java ---
@@ -401,4 +401,13 @@ public static double logsumexp(@Nonnull final double[]
arr
Github user myui commented on a diff in the pull request:
https://github.com/apache/incubator-hivemall/pull/61#discussion_r105071319
--- Diff:
spark/spark-2.1/src/main/scala/org/apache/spark/sql/hive/HivemallOps.scala ---
@@ -805,6 +805,47 @@ final class HivemallOps(df: DataFrame
1 - 100 of 549 matches
Mail list logo