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

Use ToT in HCalRecHitProducer & update the default HGCROC emulator conditions with testbeam data based info

Open tvami opened this issue 4 months ago • 3 comments

Is your feature request related to a problem? Please describe.

HCalRecHitProducer does not use ToT

Describe the solution you'd like

  • We should use ToT in HCalRecHitProducer
  • We should update the HGCROC to match the testbeam data

Additional context

https://github.com/LDMX-Software/ldmx-sw/pull/1475#issuecomment-3103641105

tvami avatar Jul 29 '25 23:07 tvami

Maybe this is a duplicate of this old issue: https://github.com/LDMX-Software/ldmx-sw/issues/1096 ?

tvami avatar Jul 30 '25 23:07 tvami

Talking with @erik-wallin we agreed that this is similar, but not exactly the same

tvami avatar Aug 27 '25 18:08 tvami

Started work on this in branch #iss1787

I'm fairly happy with the new default HCal conditons, and the reconstruction with TOT works fairly well in test beam analysis.

What remains is:

  • [ ] Double check default noise condition.
  • [ ] Verify that things are sensible also in the v14 geometry, not just the CERN prototype geometry.
  • [ ] Make use of TOT in the HcalRecProducer, following the TB analysis recipe.
  • [ ] Think about how to best store TOT calibration curve conditions.

erik-wallin avatar Oct 30 '25 22:10 erik-wallin