New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Run3 Tracker (Pixel and Strips) conditions #28441
Update Run3 Tracker (Pixel and Strips) conditions #28441
Conversation
- updated pixel local reco conditions to match updated digitizer threshold - update SiStrip local reco conditions to reflect update done in the 2018 conditions
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28441/12851
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages: Configuration/AlCa @civanch, @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @mdhildreth, @tlampen, @pohsun can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@christopheralanwest @tlampen @tocheng the GT candidates can be made full fledged GTs. For ease of book-keeping you can find the updates in this table |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28441/12903
|
The tests are being triggered in jenkins. |
Pull request #28441 was updated. @civanch, @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @mdhildreth, @tlampen, @pohsun can you please check and sign again. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
@christopheralanwest @tocheng @tlampen what is the conclusion about this GT update? |
+1 There is a summary presentation in AlCaDB meeting next Monday, but we can go ahead with this PR |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+§ |
+1 |
PR description:
The aim of this PR is to update the conditions used for the simulation of Run3 (all three years: 2021, 2023, 2024), correcting few mistakes, inaccuracies observed in the previous round of MC production in 10.6.X.
Namely:
PR validation:
Extensive validation through production of dedicated private samples has been carried out within the Tracker DPG. A full report has been gathered here.
In addition the alignment performance of the previous misalignment scenario has been checked using samples produced with the updated conditions, and a compelling reason to update it has not been found (report is available here).
if this PR is a backport please specify the original PR:
This PR is not a backport, though possibly updated GTs in 10.6.X will be requested as well for further samples production.
Cc:
@tsusa @tvami @pieterdavid @robervalwalsh @connorpa @adewit