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 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
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11159512/badge)](https://coveralls.io/builds/11159512)
Coverage increased (+1.04%) to 38.063% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11159290/badge)](https://coveralls.io/builds/11159290)
Coverage increased (+1.3%) to 38.364% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11139515/badge)](https://coveralls.io/builds/11139515)
Coverage increased (+0.9%) to 37.962% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11139395/badge)](https://coveralls.io/builds/11139395)
Coverage increased (+0.9%) to 37.948% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11122400/badge)](https://coveralls.io/builds/11122400)
Coverage increased (+1.2%) to 38.211% when pulling
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
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11121280/badge)](https://coveralls.io/builds/11121280)
Coverage increased (+0.9%) to 37.901% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11121280/badge)](https://coveralls.io/builds/11121280)
Coverage increased (+0.9%) to 37.901% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11108695/badge)](https://coveralls.io/builds/11108695)
Coverage increased (+1.2%) to 38.219% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11084626/badge)](https://coveralls.io/builds/11084626)
Coverage increased (+0.9%) to 37.916% when pulling
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
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11066675/badge)](https://coveralls.io/builds/11066675)
Coverage increased (+1.1%) to 37.855% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11050144/badge)](https://coveralls.io/builds/11050144)
Coverage increased (+1.4%) to 38.139% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/66
[![Coverage
Status](https://coveralls.io/builds/11050144/badge)](https://coveralls.io/builds/11050144)
Coverage increased (+1.4%) to 38.139% when pulling
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
[![Coverage
Status](https://coveralls.io/builds/11013730/badge)](https://coveralls.io/builds/11013730)
Coverage increased (+1.1%) to 37.846% when pulling
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
20 matches
Mail list logo