Github user takuti commented on the issue:

    https://github.com/apache/incubator-hivemall/pull/89
  
    > Adding an alias -tol for -eps is a good idea.
    
    Sure.
    
    >  it accept the input format of Hivemall's feature vector
    
    Oh, it's not format matter by the way; `String` or `FeatureValue` in terms 
of type.
    
    Current implementation repeatedly parses feature-value-formatted `String` 
at 
[HERE](https://github.com/takuti/incubator-hivemall/blob/bc4b9d3a13310d85c0fb4fc572d0338fefee0a8d/core/src/main/java/hivemall/topicmodel/AbstractProbabilisticTopicModel.java#L66-L69)
 for every time mini-batch is passed. If we directly pass a two-dimensional 
array of `FeatureValue` as word counts, we can avoid the unnecessary parse 
operations.



---
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 if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to