RTA Shift Summary 20/02/2023 - 06/03/2023
Personnel
RTA Maintainer: Rosen Matev (@rmatev)
RTA Shifter: Patrick Spradlin (@spradlin)
General
The comments of this issue were used by the RTA Shifter (@spradlin) to keep notes and help me to organize shifter activity related to MRs. They are an approximate shifter activity log.
New set of unpredictably failing tests
See #538 (closed).
The following tests in Moore occasionally fail due to mismatches with reference for 'PP2MCPRelations buffer size' counters:
- Hlt2Conf.test_hlt2_noUT_trackefflines
- Hlt2Conf.test_hlt2_trackefflines
- Hlt2Conf.test_persistreco_check_flavourtags
The impending merge of !2146 (merged) should suppress failures due to these mismatches. A permanent solution is in progress.
Allen merge pipelines
I have been seeing occasional failures to create and execute Allen merge pipelines. The failed pipelines are tagged with an 'error' label that has the hover text, 'The pipeline job activity limit was exceeded.' In at least one case, i was able to manually run the pipeline at a later time. In other cases, the pipeline was run properly on the subsequent push to the MR branch.
Open MRs
Shifter action required
In no particular order. In sets of MRs, the MR in bold is where the most recent test was run.
| MRs | Title/Description | Shifter comments |
|---|---|---|
| LHCb!3947 (closed) | Add a new interface in TransportSvc to simulate the TransportSvc.intersections function under LHCb_v50 version | Active development; build failures in last ci-test |
| !2117 (merged) | hlt2 lines and new reco for PbPB | Active development; FATAL run-time errors in last ci-test |
| LHCb!3969 (merged) Rec!3310 (merged) | Introduce (and use) EndUT state | Active development; author asked to review and comment on [last ci-test])(Rec!3310 (comment 6491564)) |
| LHCb!3935 (merged) Rec!3279 (merged) Analysis!946 (merged) DaVinci!815 (merged) | Add MCPrimaries and more MC functors | Authors pinged; expect update soon |
| Allen!1018 (closed) | no gec and veloSP lines for pit + change FT -> FTCluster name in passthrough lines | Status unknown; authors pinged |
| Allen!1083 (merged) | Generate lumi schema in Allen configuration | Tests good; assigned to reviewer |
| Allen!1079 (merged) | Add counters for random PV position to lumi summary | Further activity awaiting merge of Allen!1083 (merged) |
| Rec!3277 (merged) !2011 (merged) | Update JetAccessories to reconstruct MC jets | Active discussions on Rec!3277 (merged); merge conflicts in !2011 (merged) |
| Allen!1095 (merged) | Downstream/LongLivedParticle track reconstruction | Active development; build failures in recent tests |
| Allen!1111 (merged) !2034 (merged) | Tests to compare CPU and GPU decoding for Plume | Active development; Allen!1111 (merged) is marked Draft |
| Lbcom!652 (merged) !1703 (merged) | Fix CaloClusterEfficienncy/Use new MC samples in tests with data-like decoding versions | Follow-up with authors; probably ready for final testing; !1703 (merged) historically assigned to maintainer for some time |
| Detector!288 (merged) LHCb!3757 (merged) Rec!3107 (merged) !1758 (merged) MooreOnline!184 (closed) | Modernized Calo monitoring and dropped MoniAlg | Waiting general update of test samples to Run3; see author's comment on LHCb!3757 (comment 6467970) |
| !2122 (merged) | update according to BnoC 12Jan meeting | Active development; reviewer comments that need to be addressed; last test problematic |
| Allen!992 (merged) | Add multiplicity cuts for diphoton lines and include HLT1 Pi0 line | Follow-up; may need review; may be ready to assign to maintainer |
| LHCb!3872 (merged) Rec!3196 (merged) Allen!1072 (merged) !1909 (merged) Analysis!958 (merged) DaVinci!790 (merged) MooreOnline!190 (merged) MooreAnalysis!100 (merged) | Make name of algorithms independent of instantiation order | Active development |
| Allen!1129 (merged) !2129 (merged) | Update Allen-in-Gaudi configuration to fix tests for DD4Hep | Follow-up with author; may be ready for final testing and assignment to maintainer |
| !2132 (merged) | B&Q lines: Xib -> J/psi hhh and bbaryon -> Lc Ds h(h) | In review; comments to be resolved by authors |
| LHCb!2995 (merged) Lbcom!550 (merged) Boole!334 (merged) Rec!2403 (merged) Allen!865 (merged) Detector!181 (merged) | UT-newID-and-Geo | In review; probably need better titles |
| Rec!3274 (merged) !1873 (merged) | Sign functor, y-z straight-line intersection functors | Active development; may be ready for testing |
| Rec!3333 (merged) !2139 (merged) | Replace ProbeAndLongMatcher with MuonProbeToLongMatcher | Probably ready to be assigned to maintainer for merge; minor name change of an algorithm |
| LHCb!3907 (merged) Rec!3247 (merged) !1984 (merged) Analysis!942 (merged) | SOACollection particlev2 | Active discussions |
Assigned to maintainer
| MRs | Title/Description | Shifter comments |
|---|---|---|
| Allen!1100 (merged) | Add option to include ODIN information in ntuple monitoring | |
| !2031 (closed) | Add flags for alignment in reconstruction | Breaks a test in Alignment; maintainer delayed merging awaiting a MR that will fix the test |
| Rec!3322 (merged) DaVinci!853 (merged) | Added candidate multiplicity counter to ParticleCombiner | |
| LHCb!3988 (merged) Lbcom!653 (merged) Rec!3329 (merged) MooreAnalysis!113 (merged) Alignment!354 (merged) | Remove LinkerTable and LinkerTool, augment LinkedTo, LinkedFrom | |
| LHCb!3986 (merged) Analysis!956 (merged) DaVinci!841 (merged) | make pp2mcp relations unpacking consistent across different functions | |
| !2054 (merged) | More fixes for Hlt2 lines | Requires approval |
| !2146 (merged) | pp2mcp buffer size counter | Work-around for #538 (closed) |
Edited by Patrick Spradlin