-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
PPS: preparation for 2022 conditions #35177
Conversation
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35177/25109
|
test parameters: |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35177/25111
|
looks like a bug in filename checking part of bot. I will fix it and re-run the check here soon |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35177/25116 |
A new Pull Request was created by @jan-kaspar for master. It involves the following packages:
@malbouis, @andrius-k, @yuanchao, @kmaeshima, @ErnestaP, @ahmad3213, @cmsbuild, @jfernan2, @rvenditti, @francescobrivio, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35177/25118 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5d3ea3/18365/summary.html Comparison SummarySummary:
|
Hi @jan-kaspar,
do I understand that correctly that they conditions are not expected to change and that's why you choose them to be external root files?
Kind of the same question: won't the alignment of PPS change over time and so it should not be read from an xml file? |
Hi @tvami , the optics and alignment related to this PR reflect the current forecast for Run3 and are intended for preparatory simulations. It is almost clear that in reality both the optics and alignment will be different. These conditions will be deduced from real data and will go to DB for the use in reconstruction. Drawing from the Run2 experience, the optics is expected to be stable (for a year or so) while the alignment may evolve a little fill by fill. For the (direct) simulation, in Run2, we had to prepare a limited number of profiles (2 or 3 per year) which could reasonably reproduce the data. Given the small number of profiles we kept the optics files in the external repo and the alignment files in CMSSW. If preferable, both can be stored in DB for Run3. In Run2 we rarely (if ever) updated these conditions. |
Hi @jan-kaspar thanks for the explanation! If there are ~3 updates per year, I'd say it's already a good motivation to move to the DB. What do you @francescobrivio @malbouis think? |
OK, thanks! I will pass your recommendation to those who will take over from me next year. |
+dqm |
you'll need cms-sw/cmsdist#7282 merged before this |
+alca
|
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR contains three updates:
CalibPPS/ESProducers/python/ctppsOpticalFunctions_non_DB_cff.py
- also useful for workflows which do not consume conditions from DB.Validation/CTPPS/alignment/alignment_2022.xml
, wrt. the 2021 version, contains constants adjusted to counteract a shift recently induced by the new Run3 geometry.PR validation:
The plots below compare results before (blue solid) and after this PR (red dashed):