RTA Shift Summary 25/07/2022 - 07/08/2022
- RTA Shift Period: 25/07/2022 - 07/08/2022
- RTA Shifter: Peilian Li (@peilian)
- RTA Maintainer: Sebastien Ponce (@sponce), Rosen Matev (@rmatev), Christopher Rob Jones (@jonrob)
- Next Shifter: Christina Agapopoulou (@cagapopo)
- Details in codimd
- Issues to note:
- behaviour of Allen.mdf_input test: Allen#345 (closed)
-
iterator-vp-halves-modulestest failure in Alignment with clang build: Alignment#32 (comment 5894286) - Sometimes random failure in MooreOnline.mep_passthrough
- ramdon failure in physics efficinecy of Allen: Allen#352 (closed)
MRs in ci-test
| MR | Descriptions | status |
|---|---|---|
| Allen!689 (merged) | Implement routing bits | High priority, failure should be un-related, re-tested this morning |
| !1289 (merged) | Add D0 -> h l nu HLT2 lines |
Open Merge Requestes ci-tested with discussions/comments to follow:
| MR | Descriptions | status |
|---|---|---|
| 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!866 (merged) | Configuring the VeloOpen ODIN line | tests passed, but references update in Allen needed, should follow how to |
| Allen!889 (merged) | Modify + Add HLT1 lines for alignment | high priority, right after Allen!689 (merged), all tests passed, physics efficiency difference require reference update |
| 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 (closed) | fix throughput drop in Moore HLT1 (VELO track reconstruction) | tiny improvement in throughput, further optimization might be needed, can check with author again |
| LHCb!3670 (closed), Rec!3005 (closed), !1647 (closed) | UTDecoder ADC weighted algorithm | decrease of performance, discussion on-going |
| Rec!2979 (merged) DaVinci!716 (merged) Analysis!898 (merged) | Thor functors for MCTrackInfo | test failures to be addressed |
| Rec!1703 (merged) | Draft: Add newer minbias MC samples with up-to-date Velo DDDB tag | not for merge |
MR passed ci-test (handled to maintainer)
| MR | Descriptions | status |
|---|---|---|
| !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 | need reference updates in Moore only |
| Rec!3040 (merged) !1685 (merged) | Separate calib BXIDs from digit monitoring | |
| MooreOnline!80 (merged), Alignment!239 (merged) | add jobs for Online Alignment, start with Online alignment in ECS | |
| Rec!2985 (merged) | Update condition access for TrackMonitor and replace some MagFieldSvc by DeMagnet | one failure in Alignment requires reference update, but there are random failures align-ft-modules-prkalman and test_hlt2_all_lines_with_reco_with_streams_mdf to be further excluded |
| !1695 (merged) | update ion sequence in standalone.py |
MRs to be tested / followed:
| MR | Descriptions | status |
|---|---|---|
| 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 |
| LHCb!3528 (merged) Allen!911 (merged) !1529 (merged) | Enforce use of an encoding/decoding key (TCK) in all Hlt encoders/decoders | high priority |
| Allen!961 (merged) | Draft: Fixes and test for Allen SciFi decoding | author asks for test |
| Detector!265 (merged) | Fix station numbering in findStation to match {1, 2, 3} convention - reviewing | wait author's answer |
| 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 |
| !3024 (merged) | Draft: Functors for brem / charged calo info | More comments to be addressed, back to draft |
| LHCb!3226 (merged) | WIP: Remove usage of Run12 LHCbIDs |
Merged
| MR | Descriptions |
|---|---|
| LHCb!3699 (merged) + Detector!251 (merged) | Fixed hardcoded unit conversions between LHCb and DD4hep |
| !1625 (merged) | committed three lines for high/low pt photon + pi0 PID calibration lines |
| !788 (merged) | Lc+ -> p K- pi+, Xic+ -> p K- pi+, Xic0 -> p K- K- pi+ spectroscopy lines |
| Boole!404 (merged) | RICH monitoring tuning |
| PRConfig!240 + PRConfig!242 | add sequence for ion data |
| !1676 (merged), Lbcom!621 (merged), !1690 (merged) | Add calo efficiency checkers to all reco sequences |
| Rec!2881 (merged) | Added ABS functor |
| Rec!3044 (merged) | Update RichDetector refs for dd4hep, adapting to MR Detector/235 |
| LHCb! 3703 | Update RichDetector refs for dd4hep, adapting to MR Detector/235 |
| Detector!235 (merged) | RICH QE and Rich1 Mirror reflectivity : updating the nominal values |
| LHCb!3706 (merged) Lbcom!623 (merged) | Rich remove pd assert |
| Detector!255 (merged), LHCb!3704 (merged) | dropped unused ConditionConverter (dedicated to old xml files) (!255 (merged)) |
| Rec!3035 (merged) | Calo cleanup |
| !1691 (merged) | Apply correct db tags to qmtest |
| !1435 (merged) | follow up from starterkit |
| Detector!255 (merged) | dropped unused ConditionConverter (dedicated to old xml files) |
| Detector!252 (merged) | Fixed some harcoded units conversion in FT |
| !1523 (merged) | Early Measurement Charm production XSec update line cuts |
| Detector!253 (merged) | New unit fixes (forgotten in previous round) |
| LHCb!3701 (merged) | Fix detection of TAE data in MDF writing |
| Allen!945 (merged) | Add cache_preload.cmake to git ignore list |
| !1624 (merged) !1697 (merged) | Tests for tracking only sequence |
| Allen!874 (merged), LHCb!3452 (merged), Boole!398 (merged), Rec!2958 (merged), !1635 (merged) | First implementation of TELL40 v1 cluster output (default) and two sensor SP banks (only if FPGA clustering not available) |
| Rec!2740 (merged) | Resolve #345 "Add loki mtdocachi2 functor to thor" |
| Detector!258 (merged) | Avoid one string copy in RICH init_param call |
| Detector!257 (merged) | Reverte Detector!142 (merged) to debug |
| Rec!3046 (merged) LHCb!3708 (merged) | Update RICH qm test reference to adapt to dd4hep panel size change |
| Gaudi!1361 | forward forgotten InhibitPathes property from HiveWhiteBoard to DataSvc |
| Gaudi!1365 | Add missing include of utility |
| Gaudi!1364 | Avoid floating point exception during printing in EventSelector |
| LHCb!3709 (merged) | Make PVtracks::containers() access method const |
| Allen!943 (merged) | remove RateVsTime histograms |
| Rec!3047 (merged) | Solve the Hybrid Seeding seg fault when running on data |
| Rec!2740 (merged) | Resolve #345 "Add loki mtdocachi2 functor to thor" |
| Gaudi!1366 | Allow calling init() twice on a DataHandle |
| Allen!922 (merged) MooreOnline!130 (merged) | Only configure producers / consumers when required by algorithms in sequence |
| LHCb!3154 (merged) LHCb!3712 (merged) Rec!3049 (merged) Allen!947 (merged) !1700 (merged) Moore!5248 | Add histogram of line decision count to ExecutionReportsWriter |
| Gaudi!1367 | Update version and change log |
| Rec!3034 (merged) !1678 (merged) | change name of MuonPIDConverter to follow naming scheme |
| Allen!933 (merged), LHCb!3693 (merged), !1684 (merged) MooreAnalysis!90 (merged) | Refactor prefilters |
| Allen!946 (merged) | Create no GEC, no UT sequences for first datataking |
| Allen!730 (merged) MooreOnline!137 (merged) Allen!949 (merged) | DiMuonNoIP |
| MooreOnline!135 (merged) | Update Allen batch sizes and output threads to match pit configuration |
| PRConfig!248 | add new ion samples for lead-lead test |
| Rec!3048 (merged) | Misc. fixes for PrFTHitsChecker in dd4hep builds |
| Detector!259 (merged) | Fixes to DeVPSensor and DeVPSensorObject |
| MooreOnline!129 (merged) | Get buffers better |
| Allen!951 (merged) | Remove warning of library libHostSelecitons in standalone by moving an include.. |
| Rec!3052 (merged) | PrFTZoneHandler - Check array bounds before access |
| LHCb!3713 (merged) | ignore MagneticFieldGridReader log |
| LHCb!3710 (merged) | Fixed DD4hep load test for G4 units as ROOT stderr is different |
| Detector!261 (merged) | Fix test value for VP ladders position |
| Allen!944 (merged) | Add invert option to BX type filter |
| Detector!260 (merged) | Updated documentation on testing the description |
| LHCb!3714 (merged) | Fit MessageSvc columns width |
| Detector!264 (merged) | Remove unused function, squashes clang warning |
| Rec!3057 (merged) | update RICH Qm test ref log for dd4hep |
| Boole!382 (merged), Rec!3718 (merged): Default to the real RICH data format, Add support for RICH reading mapping versions | |
| Rec!2960 (merged) Alignment!301 (merged) MooreOnline!136 (closed) Panoptes!179 (merged) | Improve Scaling of Stereo Candidate Search in PrForwardTracking |
| LHCb!3705 (merged) Detector!254 (merged) | Improve support for Alignment workflow, Add the possibility of loading the conditions overlay from existing YAML files |
| Allen!942 (merged) | Remove limits on selected objects in SelReport maker |
| !1098 (merged) MooreOnline!133 (merged) | Add pi0 monitoring to CaloMon |
| Rec!3056 (merged) | Switch to TGeoTransportSvc in dd4hep builds |
| Rec!3055 (merged) Detector!263 (merged) | Fix layerID method in FT DD4HEP by using layerIdx |
| !1699 (merged) | Add Hlt2 and Sprucing event size qmtests |
| !1693 (merged) LHCb!3700 (merged) | Update test for new JSONSink |
| Rec!3002 (merged) | add information to VPTrackMonitor to study residuals |
| Allen!895 (merged) LHCb!3703 (merged) Detector!262 (merged) !1704 (merged) MooreOnline!140 (merged) Allen!957 (merged) | Allen muon decoding |
| Allen!954 (merged) | Add prescaler arguments |
| Rec!3059 (merged) | Fix typo in TrackMonitor.cpp that breaks LHCbPR dashboard entries |
| Rec!3022 (merged) !1666 (merged) Alignment!302 (merged) | make the PrKalmanFilter a multi transformer + Adapt configuration for change in PrKalmanFilter |
| !1714 (merged) | Make the Muon decoding / hit making configurable |
| LHCb!3711 (merged) | Add support for Velo Motion System override from YAML in PyConf |
| Rec!3002 (merged) | add information to VPTrackMonitor to study residuals |
| Rec!2818 (merged) | Class for VPHitExpectation |
| !1195 (merged) | BNV and LFV lines |
| LHCb!3697 (merged) | Add qmtexec command for running QMT tests directly |
| !1263 (merged) | Adapt to Run3 detectors for TOS fractions |
| !1651 (merged) | Add test for PV reconstruction with PatPV3DFuture |
Edited by Peilian Li