miranov25 / fastMCKalman

Fast simulation and performance parameterization - primary for ALICE3 and DUNE
MIT License
0 stars 1 forks source link

FixUpdate #19

Closed Trenchcoat95 closed 1 year ago

Trenchcoat95 commented 1 year ago

Reverted to older version of update; Added skip if update fails and added protection on z to avoid it being too large.

Old pull tests vs new: fastParticle::reconstructParticleFull: 3304 fastParticle::reconstructParticleFull: short track 2329 fastParticle::reconstructParticleFull: Too few consecutive points 51 fastParticle::reconstructParticleFull: Rotation failed 160 fastParticle::reconstructParticleFull: Propagation failed 188 fastParticle::reconstructParticleFull: Update failed 434 fastParticle::reconstructParticleFull: Too big chi2 113 fastParticle::reconstructParticleFull: Correct for material failed 8 fastParticle::reconstructParticleFull: PropagateToMirrorX failed 21 fastParticle::reconstructParticle: 2696 fastParticle::reconstructParticle: short track 2629 fastParticle::reconstructParticle: Too few consecutive points 0 fastParticle::reconstructParticle: Rotation failed 3 fastParticle::reconstructParticle: Propagation failed 21 fastParticle::reconstructParticle: Update failed 26 fastParticle::reconstructParticle: Too big chi2 2 fastParticle::reconstructParticle: Correct for material failed 15 fastParticle::reconstructParticle: PropagateToMirrorX failed 0

fastParticle::reconstructParticleFull: 3026 fastParticle::reconstructParticleFull: short track 2357 fastParticle::reconstructParticleFull: Too few consecutive points 59 fastParticle::reconstructParticleFull: Rotation failed 185 fastParticle::reconstructParticleFull: Propagation failed 259 fastParticle::reconstructParticleFull: Update failed 0 fastParticle::reconstructParticleFull: Too big chi2 124 fastParticle::reconstructParticleFull: Correct for material failed 15 fastParticle::reconstructParticleFull: PropagateToMirrorX failed 27 fastParticle::reconstructParticle: 2781 fastParticle::reconstructParticle: short track 2711 fastParticle::reconstructParticle: Too few consecutive points 0 fastParticle::reconstructParticle: Rotation failed 1 fastParticle::reconstructParticle: Propagation failed 17 fastParticle::reconstructParticle: Update failed 26 fastParticle::reconstructParticle: Too big chi2 10 fastParticle::reconstructParticle: Correct for material failed 16 fastParticle::reconstructParticle: PropagateToMirrorX failed 0

miranov25 commented 1 year ago

Could you add there also the pull log? I am merging but we should keep for reference