-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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 full simulation with beam spot information deom SimBeamSpotObjects #42664
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42664/36735
|
A new Pull Request was created by @diemort (Gustavo Gil) for master. It involves the following packages:
@cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
Until this is done, tests should be guaranteed to fail. |
-1 Failed Tests: RelVals RelVals-INPUT AddOn RelVals
RelVals-INPUT
AddOn Tests
Expand to see more addon errors ... |
Related to this PR, any news regarding cms-AlCaDB/AlCaTools#86 @francescobrivio ? |
Dear Gustavo, sorry for the delay! Things are finally moving and I have just posted on the AlCaDB CMSTalk thread the official request to include the new Cheers, |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-229cad/35618/summary.html Comparison SummarySummary:
|
Milestone for this pull request has been moved to CMSSW_14_0_X.Please open a backport if it should also go in to CMSSW_13_3_X. |
#43197 is merged, so let us test this PR again. |
please test |
I cannot see the error logs because of a HTTP 404. Does anyone can check them? |
OK thanks. Let's see if #39803 is resolved. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-229cad/35783/summary.html Comparison SummarySummary:
|
+1 it seems that errors are not related to 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. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR addresses the issue #41894 to avoid using the
BeamSpotObjects
from reconstructions and moving to the newlySimBeamSpotObjects
in the PPS full simulation. The simulation of generator protons will be based on the simulated beam spot position in accordance with the simulation workflow in central production. The correspoding tag will be provided by AlCa as given in this AlCaDB PR #86.PR validation:
The changes were validated with the current version of the PPS full simulation and replacing the tag for
BeamSpotObectsRcd
by the available preliminary tag forSimBeamSpotObectsRcd
in PrepDB here via:As shown below the vertex z position is correctly simulated by the PPS full simulation. The coordinates are centered in zero to perform the proton reconstruction within the PPS acceptance w.r.t. the TOTEM coordinates.
PR passes all code checks.
FYI @fabferro @mundim @wpcarvalho @mmusich @francescobrivio