-
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
[92x] L1T EMTF 2017 Full #20768
[92x] L1T EMTF 2017 Full #20768
Conversation
+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:
|
+1 The above mentioned commits from l1t-integration branch are already part of 94x version and part of this 92x rebased version, by construction. |
+1 |
+1 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_4_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar (and backports should be raised in the release meeting by the corresponding L2) |
@davidlange6 please merge this one |
CMSSW_9_2_X developement work is completed. Closing remaining PRs |
92X - rebased 94x EMTF PRs #19743 #20006 #20286
Once the tests pass with the inital PR:
There are a few other things related to EMTF
which are already merged to integration branch
that I should add on top the currently testing PR,
but for now, I want to make sure this one passes.
Things to add later:
v96.14
EMTF Remove include of depricated header file in Online producer that
was causing problem to HLT with git cms-checkdeps -A -a .
v96.21
EMTF: value-initialise EMTFPtLUT variables before use (13 TeV mc relvals pre8 re cycle 70x gf1 #595)
v96.28
EMTF water leak fix and 2016 / 17 conditions configurations via
hackConditions (Merge from CVS HEAD. #604).