Fix wrong DeUTSector cache of p0 parameter
It's reported, on this slide 9, that UT residual distribution are slightly different with the same MC sample but processed by DD4hep and DetDesc. One of the culprits is that the
In DD4hep,
Related talk:
Here I show a comparison of the histogram produced by TrackMonitor. This is by running a Moore job with the attached option file. +P/-P
means correct/wrong pitch size, and +S/-S
means correct/wrong strip number assignment.
Somehow I could not obtain a perfect match with an Alignment job (perhaps UT's DetDesc Alignment code is different with DD4hep Alignment code?) Anyway, I think DetDesc Alignment code will never be used, and the above comparison already shows at tracking level the discrepancies are fixed.
Validated by
-
Core Software -
RTA -
Simulation
Merge request reports
Activity
added RTA WP4 label
- Resolved by Menglin Xu
/ci-test --model lhcb-sim11
Started integration test build. Once done, check the results or the comparison to a reference build.
Many tests failed due to slight change in counters. This is expected, because this MR corrects strip number assignment during reconstruction, as if 50% UT sensors are moved by 1 strip, in comparison to ref test.
Since the MC sample used in the test contains the pitch size issue in DetDesc, it's hard to predict whether this MR will improve or worsen the counter. In any case, the counter only changes by 0.1% if not less, so not a big deal.
Edited by Hangyi Wu
added ci-test-triggered label
- [2025-02-03 17:37] Validation started with lhcb-master-mr#12434
- [2025-02-03 17:37] Validation started with lhcb-sim11-mr#226
- [2025-02-05 17:31] Validation started with lhcb-sim11-mr#237
- [2025-02-06 14:15] Validation started with lhcb-sim11-mr#242
Edited by Software for LHCbFor the record, I believe this should fix the segfault discussed in https://mattermost.web.cern.ch/lhcb/pl/9pkewya1k7faifesqj3ffq3xzy
/cc @zejia
would close the corresponding issue when the MR is approved~LHCb#382 (closed)
- Resolved by Menglin Xu
/ci-test --model lhcb-sim11