-
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
74X 25ns HLT round three #11041
74X 25ns HLT round three #11041
Conversation
A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_7_4_X. 74X 25ns HLT round three It involves the following packages: Configuration/HLT @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins. |
Pull request #11041 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
please test |
The tests are being triggered in jenkins. |
Pull request #11041 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please check and sign again. |
-1 DAS Error you can see the results of the tests here: The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
@Martin-Grunewald - do you see why this changes the default workflows? (at least thats what the PR tests say) |
@davidlange6 |
I looked at 25202 and 50202 On Sep 16, 2015, at 12:46 PM, Martin Grunewald <[email protected]mailto:[email protected]> wrote: @davidlange6https://github.com/davidlange6 — |
These 2 are fullsim MC worklfows which run the frozen HLT menus '25ns14e33_v1' or '50ns_5e33_v1', and those HLT menus are not changed in this PR. So I can only assume there is a non-reproducability issue here somewhere in CMSSW. |
Hmm, I see in the comparison GUI lots of changes unrelated to HLT (eg, JETs), and since this |
@cmsbuild, please test |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
@davidlange6 |
74X 25ns HLT round three of 25ns HLT menu updates
Based on 7410patch1
Includes previous PR #10834