TrigExPartialEB: Thread safety fix and some cleanup
- Enabled thread safety checker and fixes
- Remove obsolete boost link dependencies
- Delete old MTCalibPebOptions.py job options and MTCalib algorithm
Merge request reports
Activity
added Trigger master review-pending-level-1 labels
Hi @fwinkl,
if we're removing the old job options, we may as well remove the old class MTCalibPeb. I don't think it's useful in the new framework. I kept it during the implementation of new classes and job options for easier comparison of what's missing. I think the new version is now fully or nearly fully functional, so it may be a good time to remove the old one. We can always look up the old one in 21.1 if we need.I suggest to remove the old class unless @wiedenma has any objections?
Thanks @rbielski. Yes, I agree with that.
✅ CI Result SUCCESSAthena AthSimulation externals ✅ ✅ cmake ✅ ✅ make ✅ ✅ required tests ✅ ✅ optional tests ☁ ✅ Full details available at NICOS MR-21060-2019-02-12-14-40
✅ Athena: number of compilation errors 0, warnings 0
✅ AthSimulation: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST 33681]added review-approved label and removed review-pending-level-1 label
✅ CI Result SUCCESSAthena AthSimulation externals ✅ ✅ cmake ✅ ✅ make ✅ ✅ required tests ✅ ✅ optional tests ☁ ✅ Full details available at NICOS MR-21060-2019-02-12-15-23
✅ Athena: number of compilation errors 0, warnings 0
✅ AthSimulation: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST 33682]mentioned in commit 132fc548
added sweep:ignore label