coveralls
commented
3 years ago Pull Request Test Coverage Report for Build 948785500
- 88 of 96 (91.67%) changed or added relevant lines in 8 files are covered.
- 1 unchanged line in 1 file lost coverage.
- Overall coverage increased (+0.9%) to 80.327%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| py/dynesty/plotting.py | 6 | 8 | 75.0% | ||
| py/dynesty/bounding.py | 25 | 31 | 80.65% | ||
| <!-- | Total: | 88 | 96 | 91.67% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| py/dynesty/dynamicsampler.py | 1 | 88.01% | ||
| <!-- | Total: | 1 | --> |
| Totals | |
|---|---|
| Change from base Build 948718536: | 0.9% |
| Covered Lines: | 3683 |
| Relevant Lines: | 4585 |
Fix to not use np.random anywhere, instead rely on numpy generator. Also ensure that the map() methods get seeds to initialize the random generators, so things are fully deterministic even in the multithreaded scenario. Fixes #291