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: alignment from auto GT #26305
PPS: alignment from auto GT #26305
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26305/9004
|
A new Pull Request was created by @jan-kaspar for master. It involves the following packages: CalibPPS/ESProducers The following packages do not have a category, yet: CalibPPS/ESProducers @perrotta, @andrius-k, @kmaeshima, @schneiml, @tocheng, @cmsbuild, @franzoni, @jfernan2, @fioriNTU, @slava77, @pohsun can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
I have run AddOn tests on LXPLUS7: Could some trigger tests, please? |
please test |
The tests are being triggered in jenkins. |
I created cms-sw/cms-bot#1106 |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@cmsbuild please test to get cleaner comparisons |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
I see also a change in 136.85 (Run2018A , run 315489; red is with this PR) Is this also expected? |
To me it makes sense. The XML file currently in the release, by mistake, does not contain alignment for 2018. This alignment however has been uploaded in DB and it used with this PR. Therefore I am not surprised to see the increase of hits contributing to track reconstruction. Nevertheless, @robutti could you please check and confirm? Thanks in advance! |
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 |
Hi Jan,
sorry, I have been able to reply only now. It seems to me this is not needed any more; however, if we are really comparing tracks built without and with internal alignment, isn’t a one-entry difference too little? How large are the samples used for the checks?
Thanks, ciao.
Enrico
Inviato da iPad
Il giorno 3 apr 2019, alle ore 11:46, jan-kaspar <notifications@github.com<mailto:notifications@github.com>> ha scritto:
I see also a change in 136.85 (Run2018A , run 315489; red is with this PR)
...
Is this also expected?
To me it makes sense. The XML file currently in the release, by mistake, does not contain alignment for 2018. This alignment however has been uploaded in DB and it used with this PR. Therefore I am not surprised to see the increase of hits contributing to track reconstruction. Nevertheless, @robutti<https://github.com/robutti> could you please check and confirm? Thanks in advance!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#26305 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFF1u0pAGEHGcHV5iO278rwB2xT4AM99ks5vdHg7gaJpZM4cVgrx>.
|
PR description:
The default source of PPS alignment is changed to CondDB via auto GT.
The only two changes wrt. master are:
This PR is a replacement for the earlier-closed #26243 .
PR validation:
Below a comparison of results of 2016 data reconstruction (file /store/data/Run2016C/BTagCSV/AOD/07Aug17-v1/110000/0026FCD2-369A-E711-920C-0025905A607E.root). Blue histogram: master with alignment from XML files, red: this PR. There is perfect agreement.
Below a check on pixel reconstruction quality, re-reco performed on file /store/data/Run2017C/ZeroBias/RAW/v1/000/301/283/00000/8ED63519-2282-E711-9073-02163E01A3C6.root. The quality is maintained.