Skip to content
Snippets Groups Projects

Add optional bin labels to Histogram accumulators axes

Merged Marco Clemencic requested to merge add-axis-labels-to-histo-counters into master
All threads resolved!

With this small change we can define histograms with labels for the bins, like in:

Gaudi::Accumulators::Histogram<1, Gaudi::Accumulators::atomicity::full, int> m_trigger_lines{
  this, "lines", "lines", {5, 0, 5, "line id", {"line 1", "line 2", "line 3", "line 4", "line 5"}}};

To-do:

  • add a test

/cc @alupato

Edited by Marco Clemencic

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
  • Marco Clemencic added 3 commits

    added 3 commits

    • 9180b13e - Correctly group histograms in ROOT files
    • d611613c - Add optional bin labels to Histogram accumulators axes
    • b77dfcac - Add test of axis labels in Gaudi::Accumulators::Histogram

    Compare with previous version

  • Marco Clemencic resolved all threads

    resolved all threads

  • Marco Clemencic marked the checklist item add a test as completed

    marked the checklist item add a test as completed

  • Marco Clemencic added 5 commits

    added 5 commits

    • 6ae726fd - Take into account grouping of histograms in directories
    • db5709d9 - Test histogram writing and grouping for tools too
    • 63917b20 - Store tools' histograms in subdirs of the parent
    • a3525dd2 - Add optional bin labels to Histogram accumulators axes
    • 02169e0a - Add test of axis labels in Gaudi::Accumulators::Histogram

    Compare with previous version

  • Rebased on !1234 (merged) to avoid conflicts.

  • Marco Clemencic marked this merge request as ready

    marked this merge request as ready

  • Marco Clemencic mentioned in merge request !1236 (merged)

    mentioned in merge request !1236 (merged)

  • Sebastien Ponce approved this merge request

    approved this merge request

  • Marco Clemencic mentioned in commit b65f1d45

    mentioned in commit b65f1d45

  • Anna Lupato mentioned in merge request lhcb/LHCb!3154 (merged)

    mentioned in merge request lhcb/LHCb!3154 (merged)

  • Please register or sign in to reply
    Loading