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
enabling UL regression for 2018 (and likely Run3) #27644
enabling UL regression for 2018 (and likely Run3) #27644
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27644/11178
|
A new Pull Request was created by @Sam-Harper (Sam Harper) for master. It involves the following packages: RecoEgamma/EgammaTools @perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
I don't understand the comparison tests. There are differences in 2018 and 2026 workflows. Conditions have not yet been updated and both Run-3 and Phase-2 inherit from the previous Era. So I expected 2018, Run-3 and Run-4 workflows to switch to using the new 2017 regression, inducing reco changes in all three Eras. If this is unexpected, please resolve the issue prior to adding the updated GT candidates. |
Indeed, it is puzzling that Run3 didnt change. Something to track down. |
On closer inspection, Run3 does change. Just Run3 wasnt run in these tests which is odd. Anyways not my problem :D |
The full suite of runTheMatrix.py -l limited -i all --ibeos do pass though (which includes Run3, I also manually checked the config before and after this change and it is disabled/enabled as expected). |
@christopheralanwest good catch, I missed that in #27433. I'll submit a quick fix. |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 Changes seen only in expected workflows:
In particular, no changes are seen to pre-2018 MC or 2017 data. The number of changes in comparison tests before and after the latest commit are the same, cross-checking that no unexpected changes were introduced in the last commit. For most workflows, the number of reco changes is the same. New differences are seen in:
These correspond to the addition of changes to the There are changes to the number of reco differences in two workflows:
The changes to the number of differences in 136.85 from the last commit is again a consequence of the modified |
+1
|
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:
This PR enables the 106X UL regressions for 2018, given the imminent update of these conditions in the GT.
The 2017UL is already in the global tags so matrix tests ran locally without errors. Of course they are not fully correct yet as they are not the 2018UL corrections.
Also from looking around at the modifiers it appears Run3 activates the egamma 2018 modifier so Run3 also gets this. Which is good, they should be using this.