Github user takuti commented on the issue:
https://github.com/apache/incubator-hivemall/pull/87
@myui That's why I repeatedly tried a wide variety of combinations and
finally found `-lambda 1e-6 -tol 1e-3f -iter 512` leads successful.
What `-tol` did is to compare the absolute difference between the last and
current cumulative loss; that is, the initial implementation checked
convergence based on **absolute difference**. Meanwhile, `-cv_rate` looks
**relative difference**. Hence, in a precise sense, your commit
feb9bb132afb8d81357f43cf2ee2d94f6e0aa9b6 is not "refactor," and I need to find
a completely different optimal combination of hyper-parameters for testing.
---
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.
---
