coveralls
commented
3 years ago Pull Request Test Coverage Report for Build d8584844-e0f9-49db-a5a5-95a2bc16c22f
- 3 of 22 (13.64%) changed or added relevant lines in 2 files are covered.
- 4 unchanged lines in 2 files lost coverage.
- Overall coverage decreased (-0.03%) to 60.978%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| wandb/internal/stats.py | 0 | 2 | 0.0% | ||
| wandb/internal/tb_watcher.py | 3 | 20 | 15.0% | ||
| <!-- | Total: | 3 | 22 | 13.64% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| wandb/compat/tempfile.py | 2 | 59.09% | ||
| wandb/internal/tb_watcher.py | 2 | 21.18% | ||
| <!-- | Total: | 4 | --> |
| Totals | |
|---|---|
| Change from base Build 98cddf07-b1b4-496c-8572-8974d369cdd4: | -0.03% |
| Covered Lines: | 12914 |
| Relevant Lines: | 21178 |
Turned out to be pretty simple. It currently requires having tensorflow installed but this should cover TPU logging. Also fixed a nasty bug where we always delayed tensorboard metrics by 10 seconds and caused the process to hang for 10 seconds on finish. Now we just wait for 10 seconds from the start of the process to log events.