Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@NaokiStones Merged finally. FYI.
---
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
en
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 no
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
**Note on the performance**
For
[news20-multiclass](https://github.com/apache/incubator-hivemall/tree/master/core/src/test/resources/hivemall/classifier)
data, I have translated [o
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11159512)
Coverage increased (+1.04%) to 38.063% when pulling
**97adc5ce3d22e10
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11159512)
Coverage increased (+1.04%) to 38.063% when pulling
**97adc5ce3d22e10
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11159290)
Coverage increased (+1.3%) to 38.364% when pulling
**d781b66025385772
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11159290)
Coverage increased (+1.3%) to 38.364% when pulling
**d781b66025385772
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11139515)
Coverage increased (+0.9%) to 37.962% when pulling
**3a78282afa0faedc
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11139395)
Coverage increased (+0.9%) to 37.948% when pulling
**3a78282afa0faedc
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11122400)
Coverage increased (+1.2%) to 38.211% when pulling
**9cf6a79dbaf7dbdf
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
Manual tests both on local and EMR have been done. Could you review this?
@myui
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
Updating docs and testing on EMR...
---
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 featur
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11121280)
Coverage increased (+0.9%) to 37.901% when pulling
**7028a0f21a801242
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11121280)
Coverage increased (+0.9%) to 37.901% when pulling
**7028a0f21a801242
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@myui Thank you for review. I'll add unit test which verifies perplexity
computation, and double-check my implementation and your review later.
---
If your project is set up for it, you ca
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11108695)
Coverage increased (+1.2%) to 38.219% when pulling
**e31fc1a6bb2d4b64
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11108695)
Coverage increased (+1.2%) to 38.219% when pulling
**e31fc1a6bb2d4b64
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11108572)
Coverage increased (+1.2%) to 38.226% when pulling
**e31fc1a6bb2d4b64
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
revising at
https://github.com/apache/incubator-hivemall/compare/master...myui:HIVEMALL-91-2
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11084626)
Coverage increased (+0.9%) to 37.916% when pulling
**72edf7fc75e3dd2d
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11082936)
Coverage increased (+1.0%) to 38.018% when pulling
**864051d476cf32a1
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11082936)
Coverage increased (+1.0%) to 38.018% when pulling
**864051d476cf32a1
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11082900)
Coverage increased (+1.0%) to 38.018% when pulling
**864051d476cf32a1
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
^ cherry-picked except for unrelated modifications in AUC UDAF
---
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 projec
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@takuti Could you refine mini-batch implementation and perplexity
computation scheme based on
https://github.com/myui/incubator-hivemall/commit/cf84abc731fb00a33b7cec177850f8c026416cb0
?
-
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11068045)
Coverage increased (+1.002%) to 38.02% when pulling
**46d37b776ffd913
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11068045)
Coverage increased (+1.002%) to 38.02% when pulling
**46d37b776ffd913
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@myui Updated the package name, and, since #63 has been merged, applied
some modifications. Again, this PR is ready for review ð
---
If your project is set up for it, you can reply to
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 a
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11066675)
Coverage increased (+1.1%) to 37.855% when pulling
**75773bedffe1d21a
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11050839)
Coverage increased (+1.1%) to 37.846% when pulling
**840fa5d0b58f52e4
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11050144)
Coverage increased (+1.4%) to 38.139% when pulling
**c3807c080e09e6de
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11050144)
Coverage increased (+1.4%) to 38.139% when pulling
**c3807c080e09e6de
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11048960)
Coverage increased (+1.1%) to 37.836% when pulling
**aeec5ace0f13cde6
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11029832)
Coverage increased (+1.1%) to 37.844% when pulling
**e9c2aa05f14658a6
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
Since
[castLazyBinaryObject()](https://github.com/takuti/incubator-hivemall/blob/810f5409eba8dff131e5d3b44069fb1182fa46cc/core/src/main/java/hivemall/utils/hadoop/HiveUtils.java#L932-L939)
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11013730)
Coverage increased (+1.1%) to 37.846% when pulling
**05a13dd25cd7b7da
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[](https://coveralls.io/builds/11013730)
Coverage increased (+1.1%) to 37.846% when pulling
**05a13dd25cd7b7da
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
@myui Ready for review.
[Usage](https://gist.github.com/takuti/d24324e76d4b2ec7dc4b1d50a4d192d8) has
been updated.
---
If your project is set up for it, you can reply to this email and ha
39 matches
Mail list logo
