search

flux-framework / flux-sched

Fluxion Graph-based Scheduler
GNU Lesser General Public License v3.0
90 stars 41 forks source link

Fix match average calculation #1233

Closed milroy closed 4 months ago

milroy commented 4 months ago

The current calculation of average match times uses the total job count rather than the job count since last reset.

Use the correct count to compute the average (hence downstream calculations, too).

milroy commented 4 months ago

Setting MWP.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.1%. Comparing base (e5f6754) to head (cae3be9). Report is 169 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1233 +/- ## ====================================== Coverage 74.1% 74.1% ====================================== Files 103 103 Lines 14518 14518 ====================================== Hits 10765 10765 Misses 3753 3753 ``` | [Files with missing lines](https://app.codecov.io/gh/flux-framework/flux-sched/pull/1233?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=flux-framework) | Coverage Δ | | |---|---|---| | [resource/schema/perf\_data.hpp](https://app.codecov.io/gh/flux-framework/flux-sched/pull/1233?src=pr&el=tree&filepath=resource%2Fschema%2Fperf_data.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=flux-framework#diff-cmVzb3VyY2Uvc2NoZW1hL3BlcmZfZGF0YS5ocHA=) | `100.0% <100.0%> (ø)` | |