Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
^ Fixed and gonna be sent as a new pr soon
---
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 takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
Ack! Checking your 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 this feature
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
oops... need to fix CI errors as well.
https://travis-ci.org/apache/incubator-hivemall/jobs/226400990
---
If your project is set up for it, you can reply to this email and have your
reply
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
@takuti Revised branch is already 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
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
@myui Thanks! Are you planning to push [your revise
branch](https://github.com/myui/incubator-hivemall/tree/HIVEMALL-74) to master?
Or, can I cherry-pick them and start making additional ef
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
@takuti LGTM for other codes. Merged! Thanks.
Please send PR `[HIVEMALL-74-2]` if required.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
@takuti I think probability normalization is missing in `plsa_predict`.
```
[{"label":0,"probability":1.0},{"label":1,"probability":1.1246405E-32}]
```
---
If your project is
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
revising at https://github.com/myui/incubator-hivemall/tree/HIVEMALL-74
---
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 coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
[](https://coveralls.io/builds/11225273)
Coverage increased (+0.7%) to 38.68% when pulling
**94a8fe70088161be6
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
[](https://coveralls.io/builds/11225273)
Coverage increased (+0.7%) to 38.68% when pulling
**94a8fe70088161be6
Github user myui commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
Sure ð
---
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, or
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
@myui almost finished. could you review current code and docs?
---
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 coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
[](https://coveralls.io/builds/11066189)
Coverage increased (+0.7%) to 37.545% when pulling
**82597a2ca4ffefef
Github user coveralls commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
[](https://coveralls.io/builds/11066189)
Coverage increased (+0.7%) to 37.545% when pulling
**82597a2ca4ffefef
Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/71
Since interfaces are similar to Online LDA, this PR should be handled after
#66 is merged.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
15 matches
Mail list logo
