Skip to content
Snippets Groups Projects

Fix filling of monitoring histograms with kVec/kVecUO options.

Merged Andy Salnikov requested to merge salnikov/athena:tickets/ATR-20730 into master

Histograms with those options were not filled at all, there was a logic bug in VecHistogramFiller1D::fill method. The fix is limited to that class only and I have added a unit test for those two options. I also noticed that HistogramDef contents is not initialized and I switched to using MockHistogramDef in couple of other unit tests where I think it may be important to have properly initialized definition. And I think HistogramDef needs a default constructor which makes instances in a known state (or per-member initializers).

Fixes ATR-20730

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
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading