You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
it should prob take the hits from trackingHitList[hitsInRegion[jHit]] and also trackingHitList[hitsInRegion[kHit]] and propagate this below in the code too.
That opens up immediately that 50 in int hitsInRegion[50]; might be not realistic
Is your feature request related to a problem? Please describe.
This is a follow-up to #1407
There are several things to make the lin-reg better. List below.
Describe the solution you'd like
ldmx-sw/Ecal/src/Ecal/EcalVetoProcessor.cxx
Line 996 in e6c6935
and then
ldmx-sw/Ecal/src/Ecal/EcalVetoProcessor.cxx
Line 1028 in e6c6935
it should prob take the hits from
trackingHitList[hitsInRegion[jHit]]
and alsotrackingHitList[hitsInRegion[kHit]]
and propagate this below in the code too.int hitsInRegion[50];
might be not realisticldmx-sw/Ecal/src/Ecal/EcalVetoProcessor.cxx
Line 1018 in e6c6935
Besides the physics changes discussed above we could also consider to move away from the ROOT approach and use Eigen instead.
Describe alternatives you've considered
Keep it as it is, currently it's not really used, but I think it has potential.
The text was updated successfully, but these errors were encountered: