-
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
Standard 2024(2022,2023) Wfs For PR Testing #46880
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46880/42911
|
A new Pull Request was created by @AdrianoDee for master. It involves the following packages:
@AdrianoDee, @Moanwar, @cmsbuild, @DickyChant, @miquork, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
766d53f
to
7f1fb1f
Compare
please abort |
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46880/42912
|
Pull request #46880 was updated. @AdrianoDee, @Moanwar, @DickyChant, @miquork, @srimanob, @subirsarkar can you please check and sign again. |
-1 Failed Tests: RelVals RelVals-INPUT RelVals
RelVals-INPUT
Expand to see more relval errors ...
|
7f1fb1f
to
00f7cd4
Compare
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46880/43041
|
Pull request #46880 was updated. @AdrianoDee, @Moanwar, @DickyChant, @miquork, @srimanob, @subirsarkar can you please check and sign again. |
+1 Size: This PR adds an extra 68KB to repository
Comparison SummarySummary:
|
@cms-sw/upgrade-l2 could you have a look at this? Thanks! |
+Upgrade |
+pdmv |
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. @sextonkennedy, @antoniovilela, @rappoccio, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Given the fact that in the tests the golden wfs are quite unstable (and I'm not sure to understand why), this PR proposes to use some standard fixed lumi mask wfs for 2022,2023,2024 data wfs in the limited matrix. I took the change to add wfs for latest 2024 Eras and to have more of them in the limited matrix (removing a couple fo 2022/2023 ones).
PR validation:
runTheMatrix.py -l 145.014,145.104,145.202,145.301,145.408,145.500,145.604,145.713