coveralls
commented
3 years ago Pull Request Test Coverage Report for Build 171
- 6 of 8 (75.0%) changed or added relevant lines in 1 file are covered.
- 1 unchanged line in 1 file lost coverage.
- Overall coverage decreased (-0.01%) to 68.168%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| leaves.go | 6 | 8 | 75.0% | ||
| <!-- | Total: | 6 | 8 | 75.0% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| leaves.go | 1 | 81.46% | ||
| <!-- | Total: | 1 | --> |
| Totals | |
|---|---|
| Change from base Build 170: | -0.01% |
| Covered Lines: | 1861 |
| Relevant Lines: | 2730 |
dmitryikh
fredrikluo
The PredictSingle returns 0.0 when there is an error, which could be sometimes confusing what is going on unless you read the code.
Maybe it's a good idea to just return the explicit errors, in order to be more friendly for the client code.