RTA Shift Summary 25/05/2021 - 07/06/2021
RTA shifter: Ganrong Wang (@gawang)
RTA maintainer: Chris Jones (@jonrob)
Shift period: 25/05/2021 - 07/06/2021
Next shifter: Xiaoyu Zhu (@zhux)
Useful Imformation
1.Useful link:
(L1) https://twiki.cern.ch/twiki/bin/viewauth/LHCb/RealTimeAnalysisShifterChecklist
The "Daily Routine" in that page is most important
(L2) https://indico.cern.ch/event/855074/contributions/3596715/attachments/1926611/3189833/Training_for_RTA_software_shifters.pdf
p8-11 p19-26 important to shifter
(L3) https://twiki.cern.ch/twiki/bin/view/LHCb/LHCbNightlies#Nightly_tests_reference_files
The "Nightly tests reference files" in that page is most important for ref update-
For ref update
(1) you should comfirm the test failure caused by the ref. (2) Follow link (L3) https://twiki.cern.ch/twiki/bin/view/LHCb/LHCbNightlies#Nightly_tests_reference_files and https://docs.google.com/spreadsheets/d/1W8Z3GxA-Snw1RzKeYqRSfXshpoQa60YPROEKIH5BdKQ/edit#gid=406405759 (3) You may need these git command: > git status > git add -u > git commit -m ' information about this commit' > git commit --amend --no-edit (//if you did miss some file in the last commit use this to add them to avoid having two commits in the MR instead of one) > git reset --soft HEAD^ > git reset . or git reset <filename> -
Q&A
(1) Q: The difference between the Nightly tests? A:
(2) Q: When do I need to update the ref? (3) Q: When can I ask Maintainer to merger a MR?
Open MRs
The MRs start with * will be followed by Ao Xu (@axu).
| Ref. | Description | Status |
|---|---|---|
| !771 (merged) | Change the seeding input and output | discussion ongoing |
| !792 (merged) | Runtime sequence selection | discussion ongoing |
| !806 (merged) | Add script to produce ECAL clustering efficiency plots | check lhcb-head-2 test |
| !834 (merged) | Added new rare charm Hlt2 lines | waiting merge |
| !837 (merged) | Add test for new functionality in Rec!2443 (merged) | check lhcb-gaudi-head result |
| !856 (merged) | Add dimuon trigger lines | check ci-test result |
| !856 (merged) | Add dimuon trigger lines | check ci-test result |
| !868 (merged) | Support full paths in output file with streaming | check ci-test result |
| !872 (merged) | attempt to get the Moore tests green on the new platforms | check lhcb-lcg-test result |
| !876 (merged) | Fix replicated file names | check ci-test result |
| MooreAnalysis!34 (merged) | Add efficiency checker test | discussion ongoing |
| MooreAnalysis!33 (merged) | Add MCTupleToolTOS to HltEfficiencyChecker control flow | |
| LHCb!2875 (closed) | IUpdataManagerSvc interface | waiting for Sim10 release |
| LHCb!2931 (merged) | CMake modernization | check lhcb-new-cmake test result |
| LHCb!2939 (closed) | Adding a clone method to FTCluster | discussion ongoing |
| LHCb!2942 (merged) | Add v7 FT decoding | discussion ongoing and re-test |
| LHCb!2974 (merged) | Unset LHCb default DataType | wait for Gauss!720 (merged) |
| LHCb!2991 (closed) | MuonMVA2 in HLT | dev ongoing |
| LHCb!3022 (merged) | Dropping GaudiAlgorithm from Particle2MCLinker | dev ongoing |
| LHCb!3056 (merged) | Modernize LumiEvent | wait for LCG_100 |
| LHCb!3064 (merged) | max number of digits per sensor fixed as 252 | dev ongoing |
| LHCb!3065 (closed) | Odin Bank Run3 decoder | check ci-test result |
| LHCb!3067 (merged) | Add some v2 proxies to pr tracks | wait for Rec!2443 (merged) |
| LHCb!3074 (merged) | Make findAllSectors in UTDAQ more general | update ref file |
| LHCb!3084 (merged) | Fixed duplicated property name 'RootInTES' warnings | check Throughput |
| Lbcom!551 (merged) | CMake modernization | check lhcb-new-cmake test result |
| Lbcom!554 (merged) | Add cluster ET and area to CaloClusterEfficiency checker | check ci-test result |
| Rec!2331 (closed) | Cache for TrackMatchAlg | discussion ongoing |
| Rec!2376 (merged) | Change the seeding input and output | dev ongoing |
| Rec!2416 (merged) | CMake modernization | check lhcb-new-cmake test result |
| Rec!2428 (merged) | Return std::string for sensor name | check lhcb-dd4hep test failure |
| Rec!2432 (merged) | Fix bug SelectiveBremMatchAlg | discussion ongoing |
| * Rec!2443 (merged) | PrKalman Filter enable fitting of velo tracks | update ref |
| Rec!2445 (merged) | Adapt to LHCb!3067 (merged) | wait for Rec!2443 (merged) |
| Rec!2450 (merged) | Adapt the UT sector handling in PrAddUTHits | upstream !3074 (merged)/needs ref update |
| Phys!921 (merged) | New Particle2MCLinker | depends on LHCb!3001 (merged) |
| Phys!931 (merged) | CMake modernization | check lhcb-new-cmake test result |
| Phys!938 (merged) | follow changes in LHCb!3056 (merged) | check ci-test result |
| Allen!429 (merged) | Multi-event scheduler | wait for Allen!570 (merged) |
| Allen!514 (merged) | Use endvelo states in compassUT | discussion ongoing |
| Allen!521 (merged) | Add SelReport writer | check ci-test result |
| Allen!552 (merged) | Runtime sequence selection | discussion ongoing |
| Allen!557 (merged) | Rate reports in Allen | wait for rebase |
| Allen!561 (merged) | Read MC info from MDF file | discussion ongoing |
| Allen!567 (merged) | Support std::array in properties | dev ongoing |
| Allen!571 (merged) | Added cos(DIRA) to two-track vertex | discussion ongoing |
| Alignment!184 (merged) | CMake modernization | check lhcb-new-cmake test result |
| Alignment!185 (merged) | Adapted code to explicit geometry | dev ongoing |
| Gauss!720 (merged) | Unset LHCb default DataType | hand over to gloria |
Merged MRs
| Ref. | Description |
|---|---|
| Alignment!186 (merged) | reformulate constraint condition |
| Analysis!796 (merged) | workaround clang 11 bug by using STL over rangeV3 |
| !833 (merged) | Stream to multiple files |
| !728 (merged) | Velo cluster monitoring |
| !804 (merged) | Add new QEE HLT2 lines to master |
| !835 (merged) | Add onia sprucing lines |
| !836 (merged) | Add test for Rec!2440 (merged) |
| !841 (merged) | Add 2 option files to test all Sprucing lines |
| !845 (merged) | Add ParticleMassMonitors to bremsstrahlung test |
| !849 (merged) | Update HltEfficiencyChecker docs for MooreAnalysis!38 (merged) |
| !850 (merged) | Add B2OC sprucing lines |
| MooreAnalysis!38 (merged) | Turn on DEBUG mode in HltEfficiencyChecker |
| LHCb!3001 (merged) | Towards dropping GaudiAlgorithm from LHCb |
| LHCb!3035 (merged) | Flexible UT hits container |
| LHCb!3060 (closed) | Add assignment operator, clear() to PrUTHits |
| LHCb!3061 (merged) | Update RICH decoding for realistic cable maps from the pit |
| LHCb!3062 (merged) | Add new platforms strings to PlatformInfo |
| LHCb!3069 (merged) | Improve usage of RawBank::BankType as Gaudi::Property |
| * Rec!2406 (merged) | Overlap/correction algo speed increase |
| Rec!2422 (merged) | Change PrUTHitHandler and algorithms for SOACollection |
| Rec!2431 (merged) | Tweak VPOverlapMonitor |
| Rec!2435 (merged) | Clean and modernize VPTrackMonitor |
| Rec!2439 (merged) | Recovery for badly encoded UT raw bank |
| * Rec!2440 (merged) | Fix init of MeasurementProviderT |
| Rec!2442 (merged) | MaxNumTracks and Full VELOClusterTrackingSIMD |
| Rec!2447 (merged) | Adapt to changes in LHCb!3069 (merged) |
| Phys!939 (merged) | Add ParticleMassMonitor |
| Allen!556 (closed)(closed) | SciFi-V6 |
| Allen!570 (merged) | Fix TwoTrackPreprocess |
| Allen!573 (merged) | Fix throughput comparison |
| Allen!576 (merged) | Follow Rec!2422 (merged) and LHCb!3035 (merged) |
| Allen!577 (merged) | Adapt to LHCb!3069 (merged) and LHCb!3059 (merged) |
| Alignment!170 (merged) | Script 'gaudisplititer' |
| Alignment!182 (merged) | Update SciFi parameters |
Closed MRs
| Ref. | Description | Why |
|---|---|---|
| Allen!568 (closed) | Fix throughput comparison | Avoid unsupported CI runners |
| Rec!2437 (closed) | UT overflow segfault fixed | Replaced by Rec!2439 (merged) |
| Allen!572 (closed) | Fix throughput comparisons | Recreated to help assign the CI pipeline |
Edited by Ganrong Wang