RTA Shift summary 08/08/2022 - 21/08/2022
- RTA Shift Period: 08/08/2022 - 21/08/2022
- RTA Shifter: Christina Agapoupoulou (@cagapopo )
- RTA Maintainer:Christopher Rob Jones (@jonrob)
- Next Shifter: Andre Gunther (@gunther )
- Issues to note:
- behaviour of Allen.mdf_input test: Allen#345 (closed) , has been resolved by Allen!967 (merged)
-
iterator-vp-halves-modulestest failure in Alignment with clang build: Alignment#32 (comment 5894286) - Sometimes random failure in MooreOnline.mep_passthrough
- Moore throughput tests were not running for a few days due to problem with the Jenkins machine dedicated to them. The machine was restarted and queue was clear. Tests of MRs where a throughput impact is expected should be repeated.
Open Merge Requestes ci-tested with discussions/comments to follow:
| MR | Descriptions | status |
|---|---|---|
| LHCb!3528 (merged) Allen!911 (merged) !1529 (merged) | Enforce use of an encoding/decoding key (TCK) in all Hlt encoders/decoders | high priority, test failures, waiting for @graven to help |
| Allen!742 (merged) | Velo-SciFi track reconstruction based on SciFi seeding and Matching algorithms | high priority, Allen pipeline green, stack ci test needs reference update and fix in failed Moore test |
| LHCb!3735 (merged) | Check SciFi banks and sort them by hand | high priority, ci tests good, format fix required |
| LHCb!3717 (merged) | Remove getDet in CaloFuture2DView | warning in dd4hep build, is being followed up by the deveper and the maintainer |
| LHCb!3226 (merged) | Draft: Clean up LHCbID | Allen might be impacted by this, waiting for expert's feedback on how to proceed |
| !1289 (merged) | Add D0 -> h l nu HLT2 lines | throughput drop being investigated |
| !1720 (closed) | NOT FOR MERGE: qee_upgrade to master for testing purposes | ci test triggered for testing purposes, wait for developers to signal that this is ready for another round of tests |
| !1395 (merged) | B2TauTau HLT2 line | counter mismatch errors in Moore tests, to be followed up |
| Allen!742 (merged) | Velo-SciFi track reconstruction based on SciFi seeding and Matching algorithms | Allen pipeline green, stack ci test needs reference update and fix in failed Moore test |
| Allen!807 (merged) | Hlt1 Calo DiCluster Line | throughput drop and older ci test are fine (needs reference update) , merge conflicts, needs to be fixed by developer and then go through another round of tests |
| !1730 (merged) | Remove "Line" in line names | dump_for_standalone_allen test is failing, not reproducible locally, to be followed up |
| !1230 (merged) | Draft: Standard_particles to ThOr | ci-tests looks fine, needs new throughput test |
| LHCb!3716 (merged) | Add function to unbiase PVs | seg faults in LHCb RICH tests, throughput test didn't go through, to be followed up |
| Rec!3033 (merged) DaVinci!729 (merged) | Add functors (nPVs, nTracks, nFTClusters) for reading RecSummary | ci tests ok, re-run throughput job, wait for ok from reviewers |
| Allen!865 (merged), DDDB!79, SIMCOND!143, LHCb!2995 (merged), Lbcom!550 (merged), Rec!2403 (merged), !785 (merged), Detector!181 (merged), PRConfig!230 | UTHitHandle filled based on new UT channelID | ci-test trigger failed, wait @bcouturi for help |
| Allen!948 (merged) | Redefine the global event cut to not use UT information | throughput drop & reference updates needed for Allen, go after Allen!895 (merged) and Allen!689 (merged) |
| Rec!3050 | Draft:fix throughput drop in Moore HLT1 (VELO track reconstruction) | tiny improvement in throughput, further optimization might be needed, wait for author |
| LHCb!3670 (closed), Rec!3005 (closed), !1647 (closed) | UTDecoder ADC weighted algorithm | decrease of performance, discussion on-going |
MR passed ci-test (handled to maintainer)
| MR | Descriptions | status |
|---|---|---|
| !1077 (merged) | Multimuon HLT2 and Sprucing lines | Waiting for !1529 (merged) to be merged first |
| DaVinci!716 (merged) Rec!2979 (merged) Analysis!898 (merged) | Thor functors for MCTrackInfo | |
| Rec!3074 (merged) | Use new histos in Calo monitoring | reference update needed |
MRs to be tested / followed:
| MR | Descriptions | status |
|---|---|---|
| Rec!3081 (merged) | New functor TRACK | was asked to launch test |
| Rec!3082 (merged) | Remove prone-to-misunderstanding aliases | ask if this is ready to be tested |
| LHCb!3736 (merged) | Protect the muon decoding from unexpected data | high priority, discussion ongoing |
| Rec!2612 (merged) !1712 (merged) | Initialise covariance with hit errors in VeloKalman | needs ref. updates, should re-run ci-test after Rec!3043 (merged) merged to avoid reference conflicts |
| Allen!856 (merged) | Adapt HLT1 TwoKsLine to new event model | MR conflicts to be resolved |
| Allen!847 (closed) | Muon standalone reconstruction and MatchVeloMuon | more comments to be resolved |
| MooreOnline!114 | Add test to check passthrough sequence | should be tested with ci-test, not ready yet |
Merged
| MR | Descriptions |
|---|---|
| Allen!689 (merged) | Implement routing bits |
| Rec!3063 (merged) | Move Phys/Particle2MCTruth and Phys/LoKiPhysMC out of Analysis to Rec |
| !1719 (merged) | remove make_transposed_raw_banks |
| Rec!3067 (merged) | Centralize names helper |
| !1718 (merged) | Doc for running HLT1->HLT2 chain |
| Detector!265 (merged) | Fix station numbering in findStation to match {1, 2, 3} convention |
| SIMCOND!197 | create tag for VELO and SciFi macroscopic survey (magUp) |
| SIMCOND!196 | create tag for VELO and SciFi macroscopic survey (magDown) |
| Rec!3076 (merged) | Fix GenericParticle2PVRelator for DD4Hep |
| Alignment!286 (merged) | Update to allow Alignment to run multithreaded |
| PRConfig!252 | Add minbias and Bs->phiphi sample with data-like decoding versions (August 2022) to testfile DB |
| !1706 (merged) | Add documentation for (default) monitoring |
| Allen!961 (merged) | Fixes and test for Allen SciFi decoding |
| Allen!866 (merged) | Configuring the VeloOpen ODIN line |
| Allen!889 (merged) | Modify + Add HLT1 lines for alignment |
| Allen!968 (merged) | Enable lumi summary bank writing w/o all detectors present |
| Rec!3068 (merged) !1721 (merged) | Fit upstream tracks by PrKalmanFilter |
| Allen!967 (merged) | Fix Allen.mdf_input test |
| !1238 (merged) | Xi_b -> Xi Gamma and Omega_b -> Omega Gamma lines |
| Rec!3058 (merged) | Improve PrForwardTracking debugging experience |
| !1716 (merged) | Add no UT tracking and RICH only test |
| Rec!3043 (merged) !1698 (merged) | First commit on sensor unbiasing in PR VP |
| Rec!3040 (merged) !1685 (merged) | Separate calib BXIDs from digit monitoring |
| Rec!2985 (merged) | Update condition access for TrackMonitor and replace some MagFieldSvc by DeMagnet |
| !1695 (merged) | update ion sequence in standalone.py |
| MooreOnline!80 (merged), Alignment!239 (merged) | add jobs for Online Alignment, start with Online alignment in ECS |