-
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
[12_4_X] Adding BPH wfs to relval matrix #38495
Conversation
A new Pull Request was created by @AdrianoDee for CMSSW_12_4_X. It involves the following packages:
@jordan-martins, @bbilin, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
test parameters:
|
+1 |
please test |
-1 Failed Tests: RelVals-INPUT
RelVals-INPUT
Comparison SummarySummary:
|
- BuToJPsiPrimeKToJPsiPiPiK - Psi2SToJPsiPiPi - XiMinus_14TeV - Chib1PToUpsilon1SGamma - ChicToJpsiGamma - Restoring proper ordering, updating to 13p6
Pull request #38495 was updated. @jordan-martins, @bbilin, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen can you please check and sign again. |
test parameters:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b1eac1/25771/summary.html Comparison SummarySummary:
|
backport of #38494 |
@cms-sw/pdmv-l2 @cms-sw/upgrade-l2 is this needed for 12_4_1? |
These workflows were added for them to be included in the next release validation campaigns, and therefore we don't need them in 12_4_1. |
@kskovpen @AdrianoDee wouldn't they be desirable to check the "revert to CKF for pixel-less" thingy? |
Hi @kskovpen couldn't this help for the incoming validation production for the mkfit pixelles rollback? |
Hi All, could you remind me which JIRA is that (if any)? |
Thanks! Then I take it back. Needed for 12_4_1. |
could you sign it, then :) ? Thanks a lot. |
Gosh! I am in anarchy mode today, sorry! Signing |
+pdmv |
@AdrianoDee please, sign as upgrade :) |
+upgrade Adding bph wf to relvals |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. 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 |
This is a backport of #38397 and #38494. It adds to the the list of the upgrade wfs to be used in the release validation studies the following fragments:
Note that, if #38467 goes in, the naming, the number of events and jobs will be changed accordingly for the Xi wf.PR validation:
Tested with11715.0
,11716.0
,11717.0
,11718.0
,11719.0
.Tested with
11739.0
,11740.0
,11741.0
,11742.0
,11743.0
.