RTA Shift Summary 25/10/2021-08/11/2021
RTA shifter: Jonathan Davies (@jedavies)
RTA maintainer: Rosen Matev (@rmatev)
Shift period: 11/09/2021 - 25/10/2021
Next shifter: Jonathan Edward Davies (@ascarabo)
Open MRs:
| Description | MR(s) | status | Next Action |
|---|---|---|---|
| Moved LineTraj to Detector for DD4hep + cleanup | Detector!110 (merged) LHCb!3294 (merged) Rec!2594 (merged) Phys!1003 (merged) Boole!363 (merged) Run2Support!18 (merged) | tested | waiting comments from sim experts |
| Modernize PrPlotFTHits and use DataHandles | Rec!2590 (merged) | needs a new test | test |
| VPMonitors Patch | Lbcom!585 (merged) | waiting for review | test |
| Dev christina routingbits | Allen!689 (merged) | reviewing, run pipelines | test |
| Changes to the Calo code in preparation for DD4hep integration | Detector!104 (merged), LHCb!3282 (merged), Lbcom!583 (merged), Rec!2584 (merged), Phys!999 (merged), Allen!684 (merged), !1054 (merged), Boole!361 (merged), Run2Support!17 (merged), Gauss!766 (merged), Panoramix!107 (merged), Alignment!214 (merged) | review by Rosen and sim experts | merge |
| B2CC: Migration to ThOr functors | !1070 (merged) | needs Rec!2566 (merged) and review | test |
| RICH 1 alignment line | Allen!687 (merged) | run pipelines | test |
| Fix ThOr LifeTime bug | Rec!2566 (merged), !1065 (merged) | ongoing work | test |
| Add Calo Clusters and Photon Line | Allen!683 (merged) | run pipelines | test |
| Add v1::Particle compatibility layer to MASSWITHHYPOTHESES Functor | Rec!2583 (merged) | resolve comments | test |
| Potentially fix bug in configuration of TrackResChecker | !1015 (merged) | resolve comments | test |
| Kstarmumu HLT2 line | !1040 (merged) | resolve comments | test |
| RD common builders in the ThOr framework | !1021 (closed) | resolve comments | test |
| RetinaClusters decoding | Allen!655 (merged) | tested, ongoing work | test again |
| New Event Data Algorithms | Rec!2537 (merged) | waiting for !993 (merged) and LHCb!3241 (merged) | test all together |
| TrackMonitor and TrackVertexMonitor renewal: Old histos converted to the new ones | Rec!2565 (merged) | resolve discussions | test |
| update_CaloHypoEff | Lbcom!580 (merged) | resolve discussions | update references |
| Check for warnings during CUDA builds in CI | Allen!669 (merged) | run pipelines | test |
| Add a mue builder that applies brem correction | !1022 (merged) | waiting test to be implemented | test |
| Add v7 FT decoding | LHCb!2942 (merged), SIMCOND!142 | WIP dependency, SIMCOND update needed | test |
| Remove use of childBegin/childEnd | LHCb!3150 (merged), Gauss!759 (merged), Run2Support!16 (merged) | resolve discussion | merge |
| Muon Run3 decoding | LHCb!3133 (merged), Boole!358 (closed) | ongoing work | test |
| Clean up logic for customising PyConf.Algorithm outputs | LHCb!3182 (merged) | LHCb!3182 (merged) ongoing work by Rosen | test |
| Modernize MuonDet | LHCb!3188 (closed) | waiting for LHCb!3133 (merged) | test |
| Implement 2D measurements | Rec!2140 (merged), !983 (merged), Alignment!208 (merged) | ongoing work | test |
| track-based electron shower algorithm for improved electron PID | Rec!2319 (merged), !901 (merged) | issue with SIMCOND tags? | test |
| New pile-up corrections | Rec!2413 (merged), !908 (merged) | issue with SIMCOND tags? | test |
| 3 body vertex fitter | Allen!422 (closed) | run pipelines | test |
| CI: Add a throughput job with contracts enabled | Allen!663 (merged) | ongoing work | run pipelines |
| SciFi v4 decoding to v6 logic | Allen!645 (merged) | tests ok, decrease in throughput | test when ready |
| Draft: DD4Hep implementation of FT | LHCb!3264 (merged), Rec!2562 (merged), Detector!102 (merged) | Draft | test |
| Clustering for UT TELL40 decoding | Allen!607 (closed), !928 (closed) | run pipelines | ping reviewers |
| Assorted CI improvements | Allen!583 (closed) | finish pipelines | test |
| Generation of Gaudi wrapper algorithms for Allen algorithms | Allen!431 (closed), !609 (merged) | testsfailed, ongoing work | test again |
| Simple SV fit improvements | Allen!639 (merged) | fix pipelines | test |
| Ks-Line | Allen!551 (merged) | tests are good | wait for possible extra comments |
| make MuonID in hlt2 functional | !1141 (merged) Rec!2605 (merged) LHCb!3306 (merged) | resolve discussion | test |
| Converted TrackMuonUpgradeAlg Class to Functional Framework | Alignment!220 (merged) | Ready for test | test |
| Use Gaudi::Units::MeV in Properties | Allen!703 (merged) | resolve discussions | test |
| Expert Downstream Track Monitoring Algorithm | Rec!2365 (merged) | Review by Rosen | merge |
| Converted ParticleToTrackContainer Class to Functional Framework | Alignment!219 (merged) | Test failed | Test again |
| Use Detector_DIR to set DETECTOR_PROJECT_ROOT | LHCb!3305 (merged) | Rosen's work | merge |
| add FTTrackSelector | Rec!2602 (merged) | Ready for testing | test |
| Extend generated Config.cmake with upstream versions | LHCb!3304 (merged) | Assigned to Rosen, under validation | test |
| adjust CI configuration to build all sequences, rather than one sequence / job | Allen!696 (merged) | Ready for merge? | merge |
| Add MASSWITHHYPOTHESES Thor/Loki Comparison | !1136 (merged) | Moore failure being fixed | test |
| Created a TOS test on Allen SelReports for single track Hlt1 lines. | !1135 (merged) | Work ongoing | test |
| Fix ODIN v6 by converting from v6 to v7 when needed. | Allen!700 (merged) LHCb!3302 (merged) | Testing | merge? |
| B2OC: Add_21_B2DDH_Line | !1129 (merged) | Ignore for now | Monitor? |
| DD4Hep implementation of FT | Allen!698 (merged) | Test failed, work ongoing | Test again |
| Example of 'binding' tools to conditions and/or event data and/or other tools | LHCb!3293 (closed) | Needs Gaudi release | Test |
| RichDAQ - Decoding fixes when inactive links are present | LHCb!3300 (merged) | Testing | merge? |
| speed up thor selections with fast distancecalculator | !1055 (closed) | Discussion ongoing | Test again |
| Calo Digits Min ADC Line | Allen!697 (closed) | Discussion ongoing | Test |
| Draft: Add pid to hlt efficiency tuples | MooreAnalysis!60 (closed) | Ready for testing? | Test |
Merged MRs
| Description | MR(s) |
|---|---|
| fix: make sure hlt2_pp_thor does not configure any DVCommonBase algorithms | !1096 (merged) |
| Type-Erased Allen | Allen!685 (merged) |
| fix: remove obsolete header files, closes #225 | Rec!2595 (merged) |
| PrKalmanFilter but as a Tool! | LHCb!3247 (merged), Rec!2539 (merged), !1087 (merged) |
| calo future digit monitor changes | Rec!2591 (merged), !1083 (merged) |
| Added Allen SelReports writer to allen control flow. Added a test of same. | !1067 (merged), Allen!692 (merged), MooreAnalysis!57 (merged) |
| fix BPVDLS in thor | Rec!2589 (merged) |
| Move DecayTreeTuple to MooreAnalysis | MooreAnalysis!55 (merged), Analysis!828 (merged), Phys!961 (merged), LHCb!3117 (merged), !1093 (merged) |
| B2OC: Add 164 new lines and cleanups in B2OC code | !1071 (merged) |
| Algorithm that selects the particles used in the FlavourTagging | Phys!1002 (merged) |
| Add upstream particles needed for FlavourTagging | !1068 (merged) |
| Adapted code to non movable, non copyable counters | LHCb!3284 (merged), Rec!2586 (merged), Phys!1001 (merged) |
| Add views to the UT event model | Allen!672 (merged) |
| First round in dropping usage of GaudiAlgorithm in the Packers | LHCb!3192 (merged), Rec!2514 (merged), Boole!357 (merged), Gauss!763 (merged), Brunel!1166 (merged), Phys!978 (merged), Analysis!812 (merged), DaVinci!571 (merged), Noether!15 (merged), !968 (merged) |
| SciFi v4 decoding to v6 logic | Allen!645 (merged) |
| Do not set DETECTOR_PROJECT_ROOT environment variable | LHCb!3308 (merged) |
| Disable scheduler legacy mode in hlt2_pp_thor test | !1130 (merged) |
| Update HltEfficiencyChecker docs for running chained HLT1 and HLT2 | !1092 (merged) |
| Fix warning in Moore docs | LHCb!3301 (merged) |
| Add ability to chain HLT1 and HLT2 in 1 HltEfficiencyChecker job | MooreAnalysis!58 (merged) |
| Fix docs pipeline | !1112 (merged) |
| fix: all_nodes_and_algs function did not handle list of inputs | LHCb!3298 (merged) |
| make hlt2_pp_thor more robust to new or changed lines | !1106 (merged) |
| Mandatory --sequence option | Allen!671 (merged) |
| alternative branch for overlap checking if all particles are basics | Phys!996 (merged) |
| Minor fix to ConfigurableUsers dependencies | LbCom!587 Phys!1005 (merged) |
| Minor fixes/clean up to CMake configuration | LHCb!3297 (merged) Rec!2597 (merged) |
| Update References for: LHCb!3247 (merged), Rec!2539 (merged), !1087 (merged) based on lhcb-master-mr/3056 | LHCb!3247 (merged) Rec!2539 (merged) !1097 (merged) |
| Support for Online IO components | !991 (merged) LHCb!3240 (merged) MooreOnline!69 (merged) |
| Prepare LHCb v53r4 | LHCb!3296 (merged) |