ldmx-sw icon indicating copy to clipboard operation
ldmx-sw copied to clipboard

DQM histogram ranges too small for pileup

Open bryngemark opened this issue 7 months ago • 2 comments

Describe the bug In some cases, like the ecal total energy, the DQM histogram ranges are too small to capture the entire distribution when there are multiple electrons. This is most noticable for our 1e pileup validation histogram suite.

To Reproduce Look at validation plots of energy sums in the ecal from the it_pileup CI test.

Desired behavior Plot ranges extend to beyond the end of the 2e distributions.

bryngemark avatar May 06 '25 06:05 bryngemark

Can we do this somehow dynamic? Bc if we extend everywhere then it will be less useful for the 1e case. So what I have in mind is that we could pass the nelectrons info to DQM and have the bins based on that (and then it could be used for 3 electrons too)

tvami avatar Jun 20 '25 23:06 tvami

That's an idea. The electron counter sets the number of electrons so if that's run in the validation configs, that number can be pulled from the event header by any processor.

bryngemark avatar Jul 21 '25 14:07 bryngemark