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

Update L1 uGT menu to v5 and add payloads for GBRWrapperRcd for tracking MVA #14077

Closed
wants to merge 2 commits into from

Conversation

mmusich
Copy link
Contributor

@mmusich mmusich commented Apr 14, 2016

Summary of changes in Global Tags

RunII simulation

RunII data

Upgrade

@mmusich
Copy link
Contributor Author

mmusich commented Apr 14, 2016

please test

@cmsbuild cmsbuild added this to the Next CMSSW_8_1_X milestone Apr 14, 2016
@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/12381/console

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_8_1_X.

It involves the following packages:

Configuration/AlCa

@cmsbuild, @mmusich, @franzoni, @cerminar, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @tocheng this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@cmsbuild
Copy link
Contributor

@mmusich
Copy link
Contributor Author

mmusich commented Apr 15, 2016

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

Pull request #14077 was updated. @cmsbuild, @mmusich, @franzoni, @cerminar, @davidlange6 can you please check and sign again.

@mmusich mmusich changed the title Adding 7 payloads for GBRWrapperRcd for tracking MVA needed by CMSHLT-716 in pp HLT Global Tags Update L1 uGT menu to v5 and add payloads for GBRWrapperRcd for tracking MVA Apr 15, 2016
@mmusich
Copy link
Contributor Author

mmusich commented Apr 15, 2016

please test

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/12395/console

@cmsbuild
Copy link
Contributor

-1
Tested at: da47dc4
When I ran the RelVals I found an error in the following worklfows:
1306.0 step2

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step2_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log
----- Begin Fatal Exception 15-Apr-2016 14:35:34 CEST-----------------------
An exception of category 'FailModule' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'HLT_DoubleEle24_22_eta2p1_WPLoose_Gsf_v2'
   [2] Calling event method for module HLTL1TSeed/'hltL1sDoubleEG2210IorDoubleEG2220IorDoubleEG2417'
Exception Message:
Algorithm L1_DoubleEG_22_20, requested as seed by a HLT path, cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap
Please check if algorithm L1_DoubleEG_22_20 is present in the L1 menu
----- End Fatal Exception -------------------------------------------------

1330.0 step2

runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step2_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log
----- Begin Fatal Exception 15-Apr-2016 14:39:47 CEST-----------------------
An exception of category 'FailModule' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'HLT_DoubleEle24_22_eta2p1_WPLoose_Gsf_v2'
   [2] Calling event method for module HLTL1TSeed/'hltL1sDoubleEG2210IorDoubleEG2220IorDoubleEG2417'
Exception Message:
Algorithm L1_DoubleEG_22_20, requested as seed by a HLT path, cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap
Please check if algorithm L1_DoubleEG_22_20 is present in the L1 menu
----- End Fatal Exception -------------------------------------------------

25202.0 step2

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25/step2_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25.log
----- Begin Fatal Exception 15-Apr-2016 14:56:08 CEST-----------------------
An exception of category 'FailModule' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'HLT_DoubleEle24_22_eta2p1_WPLoose_Gsf_v2'
   [2] Calling event method for module HLTL1TSeed/'hltL1sDoubleEG2210IorDoubleEG2220IorDoubleEG2417'
Exception Message:
Algorithm L1_DoubleEG_22_20, requested as seed by a HLT path, cannot be matched to a L1 algo name in any L1GlobalTriggerObjectMap
Please check if algorithm L1_DoubleEG_22_20 is present in the L1 menu
----- End Fatal Exception -------------------------------------------------

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

@Martin-Grunewald
Copy link
Contributor

Changing the L1T menu from v4 to v5 requires an HLT update.

@mmusich
Copy link
Contributor Author

mmusich commented Apr 15, 2016

@Martin-Grunewald I assume the new v5 menu is not compatible with the current HLT in release, namely few seeds available in v4 disappeared. Is my understanding correct?

@Martin-Grunewald
Copy link
Contributor

Maybe this PR should stay at L1T v4 and a new PR for L1T v5?

@Martin-Grunewald
Copy link
Contributor

Yes - so how to deal with this? Separate PRs - one including the other?

@mmusich
Copy link
Contributor Author

mmusich commented Apr 15, 2016

@Martin-Grunewald in the end shouldn't everything be wrapped up in the same global tags?
I understood from today's operations meeting that we need this to be in by today or so... @fwyzard

@mmusich
Copy link
Contributor Author

mmusich commented Apr 15, 2016

@Martin-Grunewald I suggest you cherry-pick my commits to combine the GTs and the HLT menu. Does it sound a good option?

@Martin-Grunewald
Copy link
Contributor

@mmusich - OK, fine with me, will do so with the HLT updated to v5!

Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 15, 2016
Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 16, 2016
Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 16, 2016
@mmusich
Copy link
Contributor Author

mmusich commented Apr 16, 2016

-1
superseded by #14099

@mmusich mmusich closed this Apr 16, 2016
Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 18, 2016
Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 18, 2016
Martin-Grunewald added a commit to cms-tsg-storm/cmssw that referenced this pull request Apr 19, 2016
@mmusich mmusich deleted the 81XAddingTrackingMVAatHLT branch April 20, 2016 11:32
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