Skip to content
Snippets Groups Projects

Improvements to RateTestHandler, prepare for HLT2 test

Merged Shunan Zhang requested to merge rate-test-handler into master

Several improvements have been added to the RateTestHandler, to better demonstrate the test results. Including

  • List rates/event sizes in descending order, and highlight lines with high rates/sizes
  • Separate the histograms into different WGs, they are stacked and therefore replace the original ones.
  • Attach the monitoring.json file as well.

Another option is added for the Hlt2 rate test under development.

The updated plots are attached below.

link to DPA WP1 task lhcb-dpa/project#90.

cc @erodrigu @nskidmor @abertoli

lines_per_wg

hist_rate

hist_dst_size

hist_tot_size

Edited by Eduardo Rodrigues

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Eduardo Rodrigues
    • Resolved by Shunan Zhang

      We already had the comment that some may want to have the plots solely for a single WG. That should be in addition and not as a replacement for what you have, which I think is a great and useful overview/summary. To avoid too crowded a page you could make the per-WG plots available as links on the (main) page. What do you(s) think?

  • Eduardo Rodrigues changed title from Add some improvement to RateTestHandler, prepare for Hlt2 test to Improvements to RateTestHandler, prepare for HLT2 test

    changed title from Add some improvement to RateTestHandler, prepare for Hlt2 test to Improvements to RateTestHandler, prepare for HLT2 test

  • mentioned in merge request lhcb/Moore!1632 (merged)

  • Christoph Hasse
  • Shunan Zhang added 1 commit

    added 1 commit

    • 51c3b0fb - attach an extra page with plots per WG, apply some suggestions

    Compare with previous version

  • Shunan Zhang resolved all threads

    resolved all threads

  • Author Developer

    @maszyman @rmatev This MR is now ready after adding several suggested improvements. Could you take a final look, and approve/merge it?

  • Eduardo Rodrigues
    Eduardo Rodrigues @erodrigu started a thread on commit 51c3b0fb
  • 53 53 </ul>
    54 54 <object type="image/png" data="hist_rate.png"></object>
    55 55 <p>
    56 Distribution of rate of selection lines.
    57 Lines have rate larger than 1 kHz or 0 Hz are considered to be "problematic"
    58 and require some attention. The rates are listed below.
    56 Distribution of rate of selection lines. <br>
    57 The total distribution is shown as a stacked histogram, splitted into several histograms of WGs. <br>
  • Eduardo Rodrigues
    Eduardo Rodrigues @erodrigu started a thread on commit 51c3b0fb
  • 58 and require some attention. The rates are listed below.
    56 Distribution of rate of selection lines. <br>
    57 The total distribution is shown as a stacked histogram, splitted into several histograms of WGs. <br>
    58 The distributions per WG is attached in the html page below. <br>
    59 A line is considered to be "problematic" if it has a rate of 0 Hz
    60 or larger than 1 kHz, which requires some attention. <br>
    61 The rates of all lines are listed in a html page attached below,
    62 and those "problematic" lines are highlighted.
    59 63 </p>
    60 64 <object type="image/png" data="lines_per_wg.png"></object>
    61 65 <p>
    62 The number of selection lines per working group.
    63 "Other" category contains those lines have parsed names don't belong to any known WG.
    64 To make lines properly categorized, one must follow the naming convention,
    66 The number of selection lines per working group. <br>
    67 "Other" category contains those lines have a parsed name that doesn't belong to any known WG. <br>
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading