-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
HLT development in May (80X) #14445
HLT development in May (80X) #14445
Conversation
A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_8_0_X. It involves the following packages: Configuration/HLT @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
683ff9f
to
5ef0f10
Compare
Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @rekovic, @deguio, @fwyzard, @vanbesien, @mulhearn, @davidlange6 can you please check and sign again. |
80660cb
to
c0fcddd
Compare
Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again. |
c0fcddd
to
56d0c75
Compare
Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again. |
Pull request #14445 was updated. @perrotta, @cmsbuild, @dmitrijus, @Martin-Grunewald, @deguio, @fwyzard, @vanbesien, @davidlange6 can you please check and sign again. |
any remaining issues for this PR? |
+1 |
@davidlange6 |
eg, the HLT is not ready for the MC prod yet. Ok good to know. We wait on the release then.
|
@davidlange6 @fwyzard |
I think I was never aware of this plan/need, so I can’t comment on any change in plan.
|
@davidlange6 |
good to know the hidden dependency. will try to manage a release over the weekend |
....but please include #14336 in it! |
+1 |
@davidlange6 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
HLT development in May (80X).
Includes HLT PR #14325 as that is not yet merged.
(Based on CMSSW_8_0_7_patch2 and including Egamma migration PR #13845 as well)
Needs 2016 L1T menu v2!
Rebased to CMSSW_8_0_8