search

LDMX-Software / Ecal

Software module for digitizing, reconstructing, and vetoing within the ECal.
1 stars 0 forks source link

EcalVetoProcessor has ECAL layers hardcoded #51

Closed tvami closed 5 months ago

tvami commented 5 months ago

Issue

See here https://github.com/LDMX-Software/Ecal/blob/trunk/src/Ecal/EcalVetoProcessor.cxx#L552 all the 33s.

One should generally avoid hardcoded numbers but now that we are considering a reduced ECAL geometry, we ran into this

Solution

Take the layer number from the nEcalLayers_