New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Loop runs only shows last result #735
Comments
I just checked the raw results file and it seems like all runs are actually saved. |
Correction: Only runs that have a unique (identifier, value) pair are saved, likely caused by this code. |
When tests or suites are looped with TOTAL_LOOP_COUNT or TOTAL_LOOP_TIME, each loop would produce an independent result entry with identical identifiers (descriptions). However, the results viewer and exporters silently ignore any results with duplicate identifiers. This change combines the results from all loops into a single result entry to avoid the issue. Note: This change may cause data loss in existing result files if they have multiple results with the same identifiers. But since those multiple results were never "visible" to users in the first place, it's not a practical issue. Fixes phoronix-test-suite#735
When tests or suites are looped with TOTAL_LOOP_COUNT or TOTAL_LOOP_TIME, each loop would produce an independent result entry with identical identifiers (descriptions). However, the results viewer and exporters silently ignore any results with duplicate identifiers. This change combines the results from all loops into a single result entry to avoid the issue. Note: This change may cause data loss in existing result files if they have multiple results with the same identifiers. But since those multiple results were never "visible" to users in the first place, it's not a practical issue. Fixes phoronix-test-suite#735
Expected behaviour
If I loop a test or suite with the environment variables
TOTAL_LOOP_COUNT
orTOTAL_LOOP_TIME
, every successful test run should contribute a sample to the result graph, so I can get more confidence in the numbers (both average and stddev/spread)Actual behaviour
Only the results from the last loop of each test are shown in the graph.
For example, look at the results in this run.
The bar graph claims
N=3
for each result, but if you export it as JSON you'll see that there's >100 total test runs.See below for the
TOTAL_LOOP_COUNT=3
run that produced the above result:TOTAL_LOOP_COUNT=3
command outputThe text was updated successfully, but these errors were encountered: