RTA Shift Summary 09/12/2021 - 17/12/2021
RTA shifter: Alexandre Brea Rodríguez (@abrearod)
RTA maintainer: Rosen Matev (@rmatev) and Sebastien Ponce (@sponce)
Shift period: 09/12/2021 - 17/12/2021
All MRs, both open and merged, managed by me can be found in the Google document: https://docs.google.com/document/d/17ryaF1-c_Hs11JspzMPcRbdQxZJ3GRtRHYnRqgH1JE4/edit?usp=sharing
I also recommend to next shifters to take a look to previous shift period Google document, with an excellent template for RTA shifters: https://docs.google.com/spreadsheets/d/1W8Z3GxA-Snw1RzKeYqRSfXshpoQa60YPROEKIH5BdKQ/edit#gid=495308620.
Active MRs:
| Description | MR | Status | Next action |
|---|---|---|---|
| B2CC: Migration to ThOr functors | !1070 (merged) | ci-testing | waiting for test |
| Add ISID and ISABSID ThOr functors | Rec!2575 (merged) | ci-testing | Ask for merge if passes tests |
| Use rawbank views for sprucing | !1137 (merged) | ci-testing with LHCb!3303 (merged) and DaVinci!618 (merged) | Wait for test results |
| Use rawbank views for sprucing | LHCb!3303 (merged) | ci-testing in !1137 (merged) | Goes with !1137 (merged) |
| Adds PrConverters to convert between vectors of v1 and v2 RecVertex | Rec!2629 (merged) | Blocked by Allen#289 (closed) | Once Allen#289 (closed) is solved, ci-test it again |
| Updating SL lines with ThOr functors and adding new lines | !1151 (merged) | ci-test failing, author trying to find the problem | Ask status of investigation. |
| New track functors | Rec!2645 (merged) | ci-testing with LHCb!3232 (merged) | Ask for merge if passes tests |
| Track v3 modifications | LHCb!3232 (merged) | needed for Rec!2645 (merged) | Ask for merge if passes tests |
| Set tree name automatically in ROOT service | Allen!728 (closed) | First step to solve Allen#289 (closed). ci-testing | Important MR. Try to merge ASAP |
| Fix issues with ROOTService | Allen!729 (merged) | Next step to solve Allen#289 (closed) | Ask for rebase when Allen!728 (closed) has been merged. |
| Make MuonID in hlt2 functional | !1141 (merged) | reviewed, new commit | ask if it is ready to be tested with LHCb!3306 (merged) and Rec!2605 (merged) |
| DiMuonNoIP | Allen!730 (merged) | Pipeline failed | Ask for review. |
| Align VP Halves Validation and Reference | Alignment!195 (merged) | Error in tests remains | Wait until new commit |
| Add rare tau decay HLT2 lines | !1140 (merged) | Discussion going on | Wait |
| Create dedicated sprucing test lines | !1144 (merged) | ci-test failed | wait for solution |
| rich2 alignment line | Allen!717 (merged) | Changes suggested | wait |
| B2OC: Add 308 new lines, enhancements and fixes | !1199 (merged) | work needed | rebase after merge of !1144 (merged) |
| Converting AlignTrTools to Functional Framework | Alignment!221 (merged) | Waiting for answer | solve pipeline? |
| Muon integration into DD4hep | Allen!724 (merged) | More work expected on the beginning of next week | wait |
| Muon integration into DD4hep | LHCb!3343 (merged) | Cleaning code | wait |
| Create Allen run configuration for Allen wrappers | !609 (merged) | More progress needed | wait |
| Add v7 FT decoding | LHCb!2942 (merged) | Work ongoing | Wait |
| Assorted CI improvements | Allen!583 (closed) | work ongoing | No action |
| FEST, HLT1 PRR and beam test | Allen!650 (merged) | Big one from Allen, pipeline and merge conflicts | wait |
| Implement routing bits | Allen!689 (merged) | From our point of view it is ready | R. Aaij decides when it goes |
| RetinaClusters decoding | Allen!655 (merged) | Assigned to Rosen | - |
Merge requests with WIP-dependecy:
| Description | MR | Status | Next action |
|---|---|---|---|
| Adjusted Alignment calls to FitNode 1D variants | Alignment!208 (merged) | WIP dependecy | - |
| Track-based electron shower algorithm for improved electron PID | Rec!2319 (merged) | WIP dependecy | - |
| Add test for 2D measurements | !983 (merged) | WIP dependency | - |
| Remove environment fixes for Allen | !1170 (merged) | WIP dependency | - |
| Add endVertexPos to RecVertex | LHCb!3354 (merged) | WIP dependency | - |
Moved to draft:
| Description | MR | Status | Next action |
|---|---|---|---|
| Draft: Initialise covariance with hit errors in VeloKalman | Rec!2612 (merged) | Draft | - |
| Draft: RTA PID | !1210 (closed) | Draft | - |
| Draft: Clustering for UT TELL40 decoding | Allen!607 (closed) | Draft | - |
| Draft: Kstarmumu HLT2 line | !1040 (merged) | Draft | - |
| Draft: Initialise covariance with hit errors in KalmanVeloOnly | Allen!705 (closed) | Draft | - |
Merged (or closed) MRs:
| Description | MR | Status | Next action |
|---|---|---|---|
| Fixed Phoenix code compilation with DD4hep | Rec!2654 (merged) | merged | - |
| PrForward Maintenance and Monitoring | Rec!2657 (merged) | merged | - |
| #223: More unification using ADL | Rec!2653 (merged) | merged | - |
| Extending the options to run without MC hits to exclude also PrHitsMonitors | !1201 (merged) | merged | - |
| Renamed Hits Monitors to Checkers | Rec!2642 (merged) | merged | - |
| RICH Preparations for DD4Hep support | Boole!372 (merged) | merged | - |
| RICH Reco Preparations for DD4Hep support | Lbcom!593 (merged) | merged | - |
| RICH Preparations for DD4Hep support | Rec!2648 (merged) | merged | - |
| RICH Reco Preparations for DD4Hep support | LHCb!3355 (merged) | merged | - |
| Unify fit node and fit result code in order to template TrackFitMatchMonitor | Rec!2632 (merged) | merged | - |
| fix buffer overflow in AllenReportsToRawEvent and in UT track hits host buffer using gsl::span and safe_assign_to_host_buffer | Allen!708 (merged) | merged | - |
| Remove v2::Track -> v3::Tracks converter | Rec!2585 (merged) | merged | - |
| Change double to float in calo event model | Rec!2636 (merged) | merged | - |
| Change doubles to floats in calo event model | LHCb!3348 (merged) | merged | - |
| Use new histograms in TrackMonitor and TrackVertexMonitor | Rec!2565 (merged) | merged | - |
| Add Calo Clusters and Photon Line | |||
| Allen!683 (merged) | merged | - | |
| Hierarchical SOACollections | LHCb!3353 (merged) | merged | - |
| Hierarchical SOACollections | Phys!1027 (merged) | merged | - |
| Hierarchical SOACollections | Rec!2646 (merged) | merged | - |
| Use CUDA- or HIP-specific memcpy when compiling for GPU | LHCb!3358 (merged) | merged | - |
| Potentially fix bug in configuration of TrackResChecker | !1015 (merged) | merged | - |
| Fix and clean-up TrackResChecker | Rec!2644 (merged) | merged | - |
| Prefer template member function of non-template struct | Rec!2658 (merged) | merged | - |
| FTHitExpectation: Fix uninitialised variable warning | Rec!2659 (merged) | merged | - |
| Add SIMD versions of abs(int) and operator-(int) | LHCb!3362 (merged) | merged | - |
| Implemented support for large streams in ZSTD compression | LHCb!3360 (merged) | merged | - |
Return v1 and v2 calo objects in make_calo
|
!1215 (merged) | merged | - |
| Added lest loading DeFT, DeVP and DeRich1 | Detector!136 (merged) | merged | - |
| merged | - |