Skip to content

Minor update to BW test page

Shunan Zhang requested to merge bw-test-minor-update into master

Work towards lhcb-dpa/project#268. Link to lhcb-core/LHCbPR2HD!264 (merged). The updated webpages can be previewed here:

Addressing the following comments from @erodrigu :

Can we get a header with the slot and date, which identifies the thing, so say "Slot lhcb-head, 2023-08-15" or similar?

Now the start and end times are posted to the page so that one could know how long does the test take.

The really useful table contains lines such as

| 6.717 | 11.75475 | 38.215658 | 0.449216 | 32.157306 | 0.378001 |
|-------|----------|-----------|----------|-----------|----------|

We should really round those numbers to get something meaningful ;-).

Now all numbers in tables are rounded with 3 significant digits (some are 2 because the last digit is zero).

The table gives bandwidth in GB/s (good) but then the plots down in the page rather use MB/s. Better use GB/s everywhere (also in what gets posted via the GitLab bot, BTW).

Now use GB/s everywhere, also for the Gitlab feedback of the Sprucing throughput test. The x-axis of the BW plots now ranges in (-4. 0) (BW numbers are 0.x in terms of GB/s).

Use "s" for the unit of wall time, not "Sec". Again for consistency.

Done

Some other comments are already addressed before, some are not addressed yet and I put them in the description of the issue.

cc @rjhunter @lugrazet @nskidmor @abertoli

Edited by Shunan Zhang

Merge request reports