louwrentius / fio-plot

Create charts from FIO storage benchmark tool output
BSD 3-Clause "New" or "Revised" License
370 stars 87 forks source link

bench-fio can create empty csv files, to which fio-plot reacts with an unhelpful message #68

Closed vbrik closed 2 years ago

vbrik commented 2 years ago

When doing some quick tests on NVMe, I ran into the situation where bench-fio created empty csv files, apparently because --loginterval was too close to the duration of an individual benchmark trial.

When that happened, fio-plot threw an exception with the error message "statistics.StatisticsError: mean requires at least one data point" (in parse_raw_cvs_data of dataimport.py), which was not very helpful.

Perhaps it would be good to print a more helpful error message on this exception, or implement some sanity checking to avoid this situation?

louwrentius commented 2 years ago

Thanks for the feedback. The issue is resolved in the master branch, I believe.