coveralls
commented
3 years ago Pull Request Test Coverage Report for Build 978c5150-28bd-4f50-9adc-b7acf8d77f82
- 107 of 118 (90.68%) changed or added relevant lines in 9 files are covered.
- 1 unchanged line in 1 file lost coverage.
- Overall coverage increased (+0.1%) to 61.1%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| wandb/sdk/wandb_run.py | 16 | 18 | 88.89% | ||
| wandb/internal/meta.py | 0 | 3 | 0.0% | ||
| wandb/internal/sender.py | 58 | 64 | 90.63% | ||
| <!-- | Total: | 107 | 118 | 90.68% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| wandb/internal/sender.py | 1 | 72.77% | ||
| <!-- | Total: | 1 | --> |
| Totals | |
|---|---|
| Change from base Build 47daf650-5101-429e-90da-90023d9603d0: | 0.1% |
| Covered Lines: | 12982 |
| Relevant Lines: | 21247 |
Made restoring the best metric for keras work, added a test, fixed some bugs and refactored sender. I decided to put all of the update user process state from resume in one place. This means we issue a summary update request on resume but it seems like a small price to pay for keeping the code unified.
Todo: