Skip to content
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

L1t post ls1 customs initial push #5574

Conversation

mulhearn
Copy link
Contributor

This PR is a placeholder. This turns on the new L1 emulation for 2015, but leaves a legacy global L1 menu in place. Turning on this new L1 menu before the HLT does likewise breaks the standard work flows. This PR will be superseded by one which turns on L1 and HLT new menus simultaneously. Note that #5529 will be needed (to provide the new L1 menu as XML file). I included this here to get Jenkins to work checking all but the last bit about menus.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mulhearn for CMSSW_7_2_X.

L1t post ls1 customs initial push

It involves the following packages:

L1Trigger/L1TCalorimeter
L1Trigger/L1TCommon
SLHCUpgradeSimulations/Configuration

The following packages do not have a category, yet:

L1Trigger/L1TCalorimeter
L1Trigger/L1TCommon

@cmsbuild, @civanch, @nclopezo, @ktf, @mdhildreth can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor

-1
Tested at: 9de3c5d
When I ran the RelVals I found an error in the following worklfows:
1306.0 step1

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step1_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log

25202.0 step1

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVEST+MINIAODMC/step1_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVEST+MINIAODMC.log

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5574/1077/summary.html

@davidlange6
Copy link
Contributor

@mulhearn - is this PR obsolete?

@mulhearn
Copy link
Contributor Author

mulhearn commented Oct 7, 2014

Yes, it is obsolete...

@davidlange6 davidlange6 closed this Oct 7, 2014
@mulhearn mulhearn deleted the l1t-post-ls1-customs-initial-push branch October 18, 2015 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants