PrincetonUniversity / slurmise / 38294072938

93%

push

github

fix: Add ceil tolerance function for float noise (#137)

* fix: Allow expected and predicted resources to vary by up to 1 due to float noice

test_api:test_update_model_without_cmd_then_predict was failing
on my laptop with

    assert predicted.runtime == pytest.approx(slope * 5 + 10, rel=0.01)
    E   assert 46 == 45 ± 0.45

This was because the fit returned 45.000000000000014 which math.ceil
was rounding up to 46 causing the issue. CI doesn't have this issue
so it must get 45.0 or a hair under and ceil's to 45.

Loosened tolerance of this test to allow `predicted.runtime` to be up to 1.0 larger than `slope*5+10`

367 of 410 branches covered (89.51%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 1 file covered. (100.0%)

2763 of 2973 relevant lines covered (92.94%)

0.93 hits per line