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 update to L1T menu v5 incl. GT (80X) #14106
Conversation
A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_8_0_X. It involves the following packages: Configuration/AlCa @perrotta, @smuzaffar, @Dr15Jones, @franzoni, @cerminar, @fabozzi, @Martin-Grunewald, @rekovic, @srimanob, @cmsbuild, @fwyzard, @mmusich, @hengne, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
please test |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
+1 |
@Martin-Grunewald if you rebase the branch on top of the latest CMSSW_8_0_X, it should clean up some history and get rid of the core, pdmv and l1 signatures. |
bc04e36
to
bad8016
Compare
You really seem to think this is funny. What the hell! |
I think it's funny? I'm instead annoyed at the way people have strongly coupled unrelated stuff... |
Pull request #14106 was updated. @perrotta, @cmsbuild, @cerminar, @Martin-Grunewald, @franzoni, @fwyzard, @mmusich, @davidlange6 can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
i guess its the different commits with the same change that is creating the mess. For next time, git cherry-pick is your friend. |
OK, thanks for integrating! |
+1 |
Comparison is ready There are some workflows for which there are errors in the baseline: |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison is ready There are some workflows for which there are errors in the baseline: |
Comparison is ready There are some workflows for which there are errors in the baseline: |
HLT update to L1T menu v5 incl. GT (80X)
This PR includes the GT changes from #13893 plus the changes as in #14077
It also includes the previous HLT PR #13862 as that is not yet integrated, thus requiring again all signatures.
Rebased to CMSSW_8_0_X at 14:10 April 16th, as #13862 and #13893 are now integrated, so some fewer signatures!