MdtT0SegmentFitter - Reduce the maximal iteration
Hi everybody,
the trigger runs from time to time into the debug stream because the T0 mdt fit is stuck. That's because the segment fit takes massively many iterations to converge. Usually, it's done in 20-100 iterations, but in these cases it exceeds the large number of 1M. @smh already investigated this and proposed to reduce the maximum. Thanks a lot for it.
Test of the patch on 42K data16 events does not indicate any impact on the physics: All_quick_plots.pdf
Closes ATR-26098
Merge request reports
Activity
This merge request affects 1 package:
- MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MdtSegmentT0Fitter
This merge request affects 1 file:
- MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MdtSegmentT0Fitter/src/MdtSegmentT0Fitter.cxx
Adding @apsallid ,@goetz ,@rosati ,@pscholer ,@stavrop ,@jojungge as watchers
added 23.0 MuonSpectrometer labels
CI Result SUCCESS (hash 21638837) Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 74513] This merge request affects 1 package:
- MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MdtSegmentT0Fitter
This merge request affects 1 file:
- MuonSpectrometer/MuonReconstruction/MuonSegmentMakers/MuonSegmentMakerTools/MdtSegmentT0Fitter/src/MdtSegmentT0Fitter.cxx
Adding @apsallid ,@goetz ,@rosati ,@pscholer ,@stavrop ,@jojungge as watchers
added review-pending-level-1 label
added urgent label
CI Result SUCCESS (hash 21638837) Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 74556] added review-approved label and removed review-pending-level-1 label
added review-approved-tier0 label
mentioned in commit 1f97fdec
mentioned in merge request !64388 (merged)