RTA Shift Summary 14/07/2022 - 24/07/2022
RTA Shift Summary 14/07/2022 - 24/07/2022
RTA Shifter: Miroslav Saur (@msaur)
RTA Maintainer: Sebastien Ponce (@sponce)
Next Shifter: Peilian Li (@peilian)
Details in codimd
Things to note:
- behaviour of
Allen.mdf_inputtest: Allen#345 (closed)
Open Merge Requests
| MR | Description | status |
|---|---|---|
| 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) | ongoing development, significant drop in HLT1-CPU throughput observed |
| Allen!933 (merged), LHCb!3693 (merged), !1684 (merged) | Refactor prefilters | last ci-test 07/22, error in MooreAnalysis to be solved |
| Allen!807 (merged) | Hlt1 Calo DiCluster Line | last ci-test 07/23, compilation warning in Allen, changes in CALO related counters in Moore |
| 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 | ongoing development |
| Allen!889 (merged) | Modify + Add HLT1 lines for alignment | errors in Allen, needs rebase and new test |
| Allen!847 (closed) | Muon standalone reconstruction and MatchVeloMuon | waiting for a review |
| Allen!641 (closed) | Monitoring using Gaudi Accumulators | comments to be addressed |
| Allen!794 (merged) | Added PV conversion algorithm | comments and errors to be addressed |
| Allen!730 (merged) | DiMuonNoIP | last ci-test 07/19, waiting for a quick review |
| Allen!689 (merged) | Implement routing bits | authors pinged - no response |
| Allen!856 (merged) | Adapt HLT1 TwoKsLine to new event model | changes in tracking efficiency, ongoing discussion |
| Allen!422 (closed) | 3 body vertex fitter | no response 07/25 |
| LHCb!3697 (merged) | Add qmtexec command for running QMT tests directly | additional functionality to be implemented |
| LHCb!3687 (merged) | Change default raw_event_format to 0.5 | related errors in DaVinci to be solved |
| LHCb!3647 (closed) | Set 'EvtStoreSvc' as default PyConf event store | comments to be addressed |
| LHCb!3458 (closed) | Speed up projectenv.sh and improve it for super-project builds | authors pinged |
| LHCb!3582 (closed) | Add End RICH2 state locations to Ttrack template | authors pinged |
| LHCb!3293 (closed) | Example of 'binding' tools to conditions and/or event data and/or other tools | authors pinged |
| LHCb!3154 (merged) | Add histogram of line decision count to ExecutionReportsWriter | authors pinged |
| LHCb!3670 (closed), Rec!3005 (closed), !1647 (closed) | UTDecoder ADC weighted algorithm | ongoing discussion |
| LHCb!3669 (closed) | Added left and right sides | authors pinged |
| Rec!3038 (merged), Detector!248 (merged) | Fix use of enum struct in fmt::format | being tested in lhcb-gaudi-head, lhcb-lcg-dev3, lhcb-lcg-dev4 |
| Rec!3034 (merged), !1678 (merged) | change name of MuonPIDConverter to follow naming scheme | ci-test 07/21 ; error in Alignment (failed import) |
| Rec!2881 (merged) | Added ABS functor | commits need to be squashed manually, authors pinged |
| Rec!3027 (merged) | Add correct DD4Hep path to PrStoreFTHit | some issues about lb-format, experts pinged on that MR |
| Rec!2985 (merged) | Update condition access for TrackMonitor and replace some MagFieldSvc by DeMagnet | remaining error in test |
| Rec!2818 (merged) | Class for VPHitExpectation | several comments to be addressed |
| !1529 (merged), LHCb!3528 (merged), Allen!911 (merged), gaudi/Gaudi!1361 (merged) | require an explicit tck/encoding key to configure encoding, optional stable PersistReco locations | ci-test 07/22, build error in Allen |
| !788 (merged) | Lc+ -> p K- pi+, Xic+ -> p K- pi+, Xic0 -> p K- K- pi+ spectroscopy lines | ci-test OK, discussion about rates, to be checked and probably ready to be merged |
| !1523 (merged) | Early Measurement Charm production XSec update line cuts | ci-test 07/25 |
| !1676 (merged), Lbcom!621 (merged) | Add calo efficiency checkers to all reco sequences | ci-test 07/25 |
| !1624 (merged) | Tests for tracking only sequence | not for merge for now |
| !1605 (merged) | Include missing HLT2 lines for strange decays | HLT2 throughput decrease observed, ongoing di |
| !1238 (merged) | Xi_b -> Xi Gamma and Omega_b -> Omega Gamma lines | Ongoing development |
| !1508 (merged) | Add exclusive hadronic doubly charmed baryon lines | authors pinged |
| !1549 (merged) | make_cluster_shapes with GraphClustering | error in Moore test |
| !1293 (merged) | Neutral Charm lines (photons and pi0s) | waiting for authors |
| !1289 (merged) | Add D0 -> h l nu HLT2 lines | needs to be rebased |
| !1625 (merged) | KstG/PhiG high-PT lines for photon pid | ci-test 07/25 |
| MooreOnline!114 | Add test to check passthrough sequence | authors pinged |
| MooreOnline!80 (merged) | add jobs for Online Alignment | ready to be merged |
| Alignment!239 (merged) | start with Online alignment in ECS | ongoing development |
| Alignment!254 (merged) | Dump more info on eigenvalues and fix bug in eigenvector printing | needs additional development |
| Alignment!286 (merged) | Update to allow Alignment to run multithreaded | needs additional development |
| Boole!389 (merged) | Refactored Muon{Background,Digitization,DigitChecker} to work with both DetDesc and DD4hep | needs Gaudi release |
| Boole!382 (merged) | Default to the real RICH data format | errors in test |
| Boole!335 (merged) | SciFi simulation effective noise model and spillover | authors pinged |
| Boole!366 (merged) | FT new crosstalk model | authors pinged |
Merged
| MR | Description |
|---|---|
| Allen!931 (merged), Rec!3018 (merged), Boole!402 (merged), LHCb!3679 (merged), Detector!239 (merged) | Changed name of FT methods to make software clearer |
| Allen!861 (merged), LHCb!3572 (merged) Detector!202 (merged) | Decoding FT v7 in Allen and fixes to CPU v7 |
| Allen!936 (merged), MooreOnline!121 (merged) | Monitoring infrastructure, line monitoring |
| Allen!930 (merged) | Remove assert from pv beamline peak. |
| Allen!937 (merged) | Allow external projects to be fetched from preexisting dirs |
| Allen!934 (merged) | Homogeneize copy, improve error message of cast |
| Allen!935 (merged) | Fix raw bank alignment |
| Allen!926 (merged) | RICH alignment line pre/post scalers |
| Allen!907 (merged) | DumpMuonTable - include DD4hep/GrammarUnparsed.h |
| Allen!837 (merged) | Add some information about tracks and SVs to StdInfo |
| LHCb!3650 (merged) | Update RICH numbering scheme in PDPanels initialisation. |
| LHCb!3694 (merged) | Rich remove unneccessary default ctors |
| LHCb!3622 (merged), Rec!2951 (merged), !1567 (merged), !1681 (merged), Rec!2908 (merged) | add MuonTileID |
| LHCb!3675 (merged) | Drop unwanted dependency on Gen/DecFiles |
| LHCb!3681 (merged) | Check calo bank size |
| LHCb!3680 (merged) | Replace getDet with ConditionAccessor in MCReconstructible, for DD4Hep |
| LHCb!3685 (merged), Detector!242 (merged) | exclude RICH gas parameter printout from test comparisons |
| Rec!3025 (merged), LHCb!3684 (merged) | Remove local RICH workarounds for dd4hep limitations |
| LHCb!3607 (merged) | Pass std::any to dd4hep condition storage, to allow better type checking on LHCb side |
| LHCb!3632 (merged) | Tidying the FTRawBankDecoder for future work |
| LHCb!3673 (merged) | Allow updates of Velo motion system from YAML (new conditions DB) |
| LHCb!3683 (merged) | Fix LbDD4hep test_load_det_elems.py |
| LHCb!3671 (merged), Rec!3007 (merged), !1649 (merged) | Add writing and reading of RecSummary |
| LHCb!3678 (merged) | Modernize AddressKillerAlg |
| Lbcom!622 (merged) | Adapt derived condition location for DD4Hep in VPClusterMonitors |
| Rec!3030 (merged) | Changes in Charge and IsMuon for FlavourTagging relations |
| Rec!3037 (merged), !1680 (merged) | Add spectrum flag to cell histos |
| Rec!3039 (merged) | LHCb::Track::Range instead of LHCb::Tracks in VPTrackMonitor |
| Rec!3032 (merged) | Fix ASYM functor |
| Rec!3028 (merged) | Repair histogram indexing in FT monitors |
| Rec!3026 (merged) | fix(functors) ValueOr (followup from Rec!2992 (merged)) |
| Rec!3017 (merged), DaVinci!725 (merged) | Add flight distance and impact parameter functors wrt all pvs |
| Rec!3023 (merged) | Update PrimaryVertexChecker for correct beamspot position access |
| Rec!3021 (merged) | Fix name confusion in PrKalmanFilterTool LHCbID finding |
| Rec!3020 (merged) | Fix noutliers in TrackMonitor |
| Rec!2902 (merged) | fix: FunctorFactory use gaudi/Gaudi!1338 (merged) |
| Rec!3019 (merged) | refactor(functors) consolidate ARGS and FORWARDARGS into single functor |
| Rec!2992 (merged), Analysis!902 (merged) DaVinci!720 (merged) | refactor(functors) remove invalid_value |
| Rec!3000 (merged) | FT monitoring TAE mode for NZS and ZS monitorin |
| !1682 (merged) | Adding track resolution checkers to options file |
| !1683 (merged), MooreOnline!124 (merged) | Optional split clusters monitoring |
| !1679 (merged) | Make chi2 of TBTC configurable |
| !1581 (merged), MooreAnalysis!89 (merged) | Add option to exclude incompatible lines in run_moore |
| !1667 (merged) | avoid potentially huge (and inefficient) regex |
| !1669 (merged) | removing number of events from hlt1_reco_allen_track_reconstruction |
| !1630 (merged), lhcb-datapkg/PRConfig!236 (merged) | Add an option file to run Sprucing throughput test |
| !1668 (merged) | refactor(testing) remove a few old exclusions that were quite costly |
| !1661 (merged) | Update to new functor syntax |
| !1663 (merged) | Fix for Z+jet sprucing lines |
| !1665 (merged) | Speed up qm test by 10 minutes |
| !1664 (merged) | Update standard loki-thor functor table to add CL |
| !1636 (merged) | Default monitoring histograms for Hlt2 lines |
| MooreOnline!123 (merged) | Add script to download HLT1 data (in small amounts) |
| MooreOnline!120 (merged) | HLT2 online throughput test |
| MooreOnline!118 (merged) | update Allen configuration to the one used in the pit |
| MooreOnline!116 (merged) | Updated name of TrackPV2HalfMonitor |
| MooreAnalysis!81 (merged), !1545 (merged) | Fixes to non-retina cluster HltEfficiencyChecker examples and other test tidy-ups |
Back to WIP
| MR | Description |
|---|---|
| Allen!922 (merged) | Only configure producers / consumers when required by algorithms in sequence |
| Allen!853 (closed) | HLT1 lines to handle Error Banks from each subdetector |
| LHCb!3696 (merged) | feat(HltControlFlowMgr) sort timing table and introduce a row that shows the sum of all algorithms |
| Boole!367 (merged) | FT: add basic options file |
Closed
| MR | Description |
|---|---|
| LHCb!3688 (closed) | Adds a flag to enable json test validator |
| Allen!939 (closed) | Fix prefilters application order |