-
Notifications
You must be signed in to change notification settings - Fork 4
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
input files for unit test of L1TGlobalProducer
#13
input files for unit test of L1TGlobalProducer
#13
Conversation
A new Pull Request was created by @missirol (Marino Missiroli) for branch master. @smuzaffar, @epalencia, @aandvalenzuela, @iarspider, @cmsbuild, @rekovic, @cecilecaillol can you please review it and eventually sign? Thanks. |
test parameters: |
please test |
-1 Failed Tests: RelVals RelVals-INPUT AddOn RelValsRelVals-INPUTAddOn Tests
Expand to see more addon errors ...
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6a0f62/28457/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
+l1 |
+1 |
We are missing a signature here, and this cms-data PR needs (well, needed) to be merged before cms-sw/cmssw#39832 . I guess it will still be okay if this is done before the next IB is built. By the way, since the actual L1T bugfix (cms-sw/cmssw#39464) is yet to be backported to 12_5_X, I would backport "cms-sw/cmssw#39464 + cms-sw/cmssw#39832" together, unless there are objections from ORP or L1T (@cecilecaillol , @epalencia). If so, this |
merge |
Input files required for the new unit test introduced in cms-sw/cmssw#39832.
Attn: @elfontan