RTA Shift Summary 06/02/2023 - 20/02/2023
RTA Shift Summary 06/02/2023 - 20/02/2023
RTA Shifter: Ahmed Abdelmotteleb (@ahabdelm)
RTA Maintainer: Rosen Matev (@rmatev)
General:
- Temporarily fixed problems with reference updates due to platform change (DD4HEP switch)
- Fixed recurring build warning in MooreAnalysis
- Fixed problems with Allen gitlab ci (
permission deniederror)
Open Merge Requests
| MR | Description | status |
|---|---|---|
| Allen!806 (merged) | More standard way to find libclang | ci-test 16/02 --> ready for merge if test warning is to be ignored |
| Allen!1018 (closed) | no gec and veloSP lines for pit + change FT -> FTCluster name in passthrough lines | ci-test 14/02 --> errors in the hlt1 persistency tests --> more investigations required |
| Allen!1079 (merged) | Add counters for random PV position to lumi summary | to be merged after Allen!1083 (merged) |
| Allen!1083 (merged) | Generate lumi schema in Allen configuratio | ci-test 15/02, some errors in MooreOnline, author pinged and is investigating |
| Allen!1088 (closed) | report throughput on gitlab instead of mattermost | authors pinged 20/02 |
| Allen!1100 (merged) | Add option to include ODIN information in ntuple monitoring | ci-test 17/02 --> tests looks reasonable --> moved to merge |
| Allen!1096 (merged) | Addition of Run-3 Drell-Yan dimuon lines | pinged authors 13/02 |
| Allen!1119 (merged) | Line monitor documentation | authors pinged 16/02 |
| LHCb!3935 (merged) DaVinci!815 (merged) Rec!3279 (merged) Analysis!946 (merged) | Add MCPrimaries and more MC functors | ci-test 16/02 on Rec!3279 (merged) --> weird errors in Moore --> under investigation by Maintainer |
| LHCb!3757 (merged) Detector!288 (merged) Rec!3107 (merged) !1758 (merged) MooreOnline!184 (closed) | Dropped old CaloFuture2Dview and replace it with CellIDHistogram | ongoing development |
| LHCb!3591 (merged) lhcb-conddb/SIMCOND!187 (merged) lhcb-conddb/DDDB!105 (merged) Rec!2917 (merged) Detector!319 (merged) | SciFi mat contraction calibration conditions | ci-test 16/02 --> errors, authors pinged |
| LHCb!3872 (merged) Rec!3196 (merged) Allen!1072 (merged) !1909 (merged) DaVinci!790 (merged) MooreOnline!190 (merged) MooreAnalysis!100 (merged) Alignment!333 (merged) | Make name of algorithms independent of instantiation order | ongoing development |
| LHCb!3947 (closed) | Add a new interface intersections_new in TransportSvc to simulate the TransportSvc.intersections function under LHCb_v50 version | author pinged 13/02 and 20/02, ci-test started 20/02 (needs follow up) |
| Lbcom!644 (merged) Boole!439 (merged) | Added VPClusterEffSimDQ to make LHCbPR histograms | ci-test 15/02 --> expected errors, assigned to Maintainer to merge |
| Rec!3107 (merged) | Modernized Calo monitoring and dropped MoniAlg | ongoing development |
| Rec!3292 (merged) !2094 (merged) DaVinci!836 (merged) Analysis!952 (merged) | Add more functors to access neutral CALO information | ci-test 15/02 on !2094 (merged) --> final checks for Rec and Moore, other MRs are approved --> should merge as soon as author confirms everything is ready to go |
| Rec!3277 (merged) !2011 (merged) | Update JetAccessories to reconstruct MC jets | ci-test 17/02 --> some errors in Moore, pinged author |
| !1746 (merged) | BnoC: new HLT2 and Sprucing lines | ci-test 15/02 --> ready for merge, assigned Maintainer |
| !2046 (merged) | line output control flow per stream | ci-test 17/02, changes added 19/02, needs new test |
| !2026 (merged) | NOPID configuration flags for selection | ci-test 17/02 --> ready for merge, assigned Maintainer |
| !1989 (merged) | New QEE HLT2 and Sprucing lines and improvements to existing lines | ci-test 15/02 OK --> ready for merge, assigned Maintainer |
| !2031 (closed) | Add flags for alignment in reconstruction | ongoing discussions |
| MooreOnline!174 (merged) | Full Hlt2 config for data-taking | author pinged 15/02 |
| LHCb!3961 (merged) | Remove throw in HltRoutingBitsCombiner the case HLT2 data are processed with HLT2 | ci-test 16/02 --> some errors |
| !2052 (merged) | Add routing bits filter to control flow | ci-test 17/02 --> some errors, pinged author |
| LHCb!3796 (merged) | Add a new File Summary Record implementation for Run3 in JSON | fixed errors, ci-test 16/02 OK, ready for merge, Maintainer assigned |
| LHCb!3970 (merged) | Unify relation unpackers with the rest of the unpackers | ci-test 17/02 OK, ready for merge, Maintainer assigned |
| !2078 (merged) | Streaming documentation | ready for merge, Maintainer assigned |
| Rec!3233 (merged) | Adds functor for brem-corrected momenta / covariance via a wrapper | ci-test 18/02 OK, needs ref update, ready for merge, Maintainer assigned |
| !2090 (closed) !2081 (merged) | run Hlt2 including all lines on data | !2081 (merged) had a conflict; fixed --> ci-test 20/02 (needs follow up) |
| LHCb!3976 (merged) | Update PlatformInfo with several new platforms | ci-test 18/02 OK, ready for merge, Maintainer assigned |
| !2106 (merged) | BandQ: fix PID_P cut for detached_protons_tights | ci-test 18/02 OK, ready for merge, Maintainer assigned |
| Rec!3317 (merged) | add missing const in some PrMCTools scripts | ci-test 18/02 OK, ready for merge, Maintainer assigned |
| Allen!1095 (merged) | full implementation of downstream / long lived particle track reconstruction at HLT1 level | ongoing discussions |
| !2101 (merged) | Switch some cuts from 0 to None | ongoing discussions |
Merged
| MR | Description |
|---|---|
| MooreOnline!154 (merged) | Add some checks for valid MEPs |
| Rec!3306 (merged) | Remove WARNING message from WeightedRelTableAlg
|
| LHCb!3946 (merged) lhcb-datapkg/PRConfig!278 (merged) !2059 (merged) Panoptes!230 (merged) MooreAnalysis!107 (merged) DaVinci!829 (merged) MooreOnline!202 (merged) | Correctly handle geometry and conditions versions from TestFileDB |
| LHCb!3954 (merged) Rec!3311 (merged) !2091 (merged) | Two small improvements to FTRawBankDecoder |
| LHCb!3950 (merged) MooreOnline!201 (merged) | Fix noBanksCounter in Calo decoding |
| LHCb!3891 (merged) Rec!3192 (merged) | Change hitpattern such that it counts hits per layer |
| gaudi/Gaudi!1372 (merged) gaudi/Gaudi!1401 (merged) DaVinci!809 (merged) | Dropped old CaloFuture2Dview and replace it with CellIDHistogram |
| LHCb!3943 (merged) !2023 (merged) | Make calo ADC and Digit packing compatible with other packed classes |
| Allen!1116 (merged) | Fix eta calculation for backward VELO tracks |
| Allen!1052 (merged) | UT search window bug fix |
| Allen!1107 (merged) | Add fromSignal flag to MCParticle |
| Allen!1053 (merged) | Fix ECal Lumi counters and add some histograms (off by default) |
| !2075 (merged) | Hlt1EfficiencyChecker Documentation bugfix |
| LHCb!3959 (merged) | Fix the string replacement pattern in in LHCbConfigUtils.cmake |
| Rec!3234 (merged) | use FastMomentaEstimation tool for VeloMuon tracks |
| Allen!1116 (merged) | Fix eta calculation for backward VELO tracks |
| !1289 (merged) | Add semileptonic charm decay lines |
| LHCb!3956 (merged) | Fill relation tables for IsPhoton, IsNotH, ClusterMass and CaloClusterCode |
| MooreOnline!154 (merged) | Add some checks for valid MEPs |
| !1293 (merged) | Add some neutral Charm lines (photons and pi0s) |
| !1991 (merged) | Implement Monitoring of Ds mass for sprucing |
| Allen!1120 (merged) | Added the steps to create and view a callgrind profile of Allen to the documentation |
| !1508 (merged) | Add exclusive hadronic doubly charmed baryon lines based on Run-II implementations |
| !2018 (merged) | Add 2 b2cc lines with photons in the final states |
| Allen!1122 (merged) | Remove the redundant get_input function from DiMuonNoIPLine selection |
| Rec!3307 (merged) | Update the ThOr functors descriptions for the documentation |
| Alignment!344 (merged) | Add missing parts to be able to run alignment with DD4hep |
| Allen!1098 (merged) | Fix Seeding&Matching mass bias first observed in 2022 data-taking traced back to the qop parametrization |
| !2085 (merged) | Add possibility to require deployed key in production |
| !2103 (merged) | Fix output name file of hlt2_light_reco_pr_kf_without_UT_with_mcchecking.py
|
| !2096 (merged) | Increase tolerance for some CALO counters |
| !2089 (merged) | Fix test race conditions that were causing frequent test failures |
| !2082 (merged) | Make default refs symlinks to detdesc refs as a temporary fix for failing dd4hep tests |
| !2079 (merged) | Fix documentation for running HLT1 |
| !2077 (merged) | Update v3 detdesc refs |
| !2076 (merged) | Fix documentation jobs in pipeline |
| Rec!3314 (merged) | Fix BegRich2->AtT parameterised scatter |
| Rec!3302 (merged) !2060 (merged) !2100 (merged) | Adding option of default value to functor for relation tables |
| Allen!1121 (merged) | Fix issues with CI pipelines |
| Alignment!345 (merged) | Improve SciFi alignment tests |
| LHCb!3971 (merged) | Disable python hash randomization |
| Rec!3308 (merged) | Add DetDesc specific ref for PlumeReco.plume_decoding test |
| LHCb!3964 (merged) | Add DetDesc specific ref for HltDAQ.lumi_decoding test |
| LHCb!3962 (merged) | Fix detdesc tests in LHCb and Rec |
| Rec!3287 (merged) | Use ConditionAccessor in DVCommonBase rather than calling retrieveObject from detSvc |
Back to WIP
| MR | Description |
|---|---|
| Allen!1059 (closed) | calo track monitoring |
| Allen!1091 (merged) | Add lumi counter for number of muon tracks |
| Allen!794 (merged) | Added PV conversion algorithm |
| Allen!1111 (merged) !2034 (merged) | Resolve "Tests to compare CPU and GPU decoding for Plume" |
| !1770 (merged) | Remove V0 monitoring lines and add prefilter on HLT1 decision on updated Ks lines |
| !2086 (merged) | Reenable packing checks |
| !1703 (merged) | Update default_input_and_conds_hlt2 and mdf_input_and_conds_hlt2 to lastet MC simulation |
| LHCb!3972 (merged) | Improve LinksByKey implementation |
Closed
| MR | Description |
|---|---|
| Alignment!293 (closed) | Add missing parts to be able to run alignment with DD4hep |