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

add fillDescriptions function needed by the HLT #14039

Merged
merged 3 commits into from Apr 18, 2016

Conversation

mmarionncern
Copy link

This PR only adds the fillDecription function for two modules that are needed by the HLT.

Changes expected in physics results and performances : none

Packages touched :

  • JetMETCorrections/Type1MET

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mmarionncern for CMSSW_8_0_X.

It involves the following packages:

JetMETCorrections/Type1MET

@cmsbuild, @davidlange6 can you please review it and eventually sign? Thanks.
@rappoccio, @ahinzmann, @TaiSakuma, @jdolen, @nhanvtran, @schoef, @mariadalfonso 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

@fwyzard
Copy link
Contributor

fwyzard commented Apr 13, 2016

please test

@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

-1
Tested at: 26d7276
When I ran the RelVals I found an error in the following worklfows:
134.911 step3

runTheMatrix-results/134.911_RunSinglePh2015D+RunSinglePh2015D+HLTDR2_25ns+RECODR2_25nsreHLT+HARVESTDR2_25nsreHLT/step3_RunSinglePh2015D+RunSinglePh2015D+HLTDR2_25ns+RECODR2_25nsreHLT+HARVESTDR2_25nsreHLT.log
----- Begin Fatal Exception 13-Apr-2016 14:39:33 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

135.4 step3

runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step3_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log
----- Begin Fatal Exception 13-Apr-2016 14:41:22 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

1306.0 step3

runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step3_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log
----- Begin Fatal Exception 13-Apr-2016 14:43:20 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

1330.0 step3

runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step3_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log
----- Begin Fatal Exception 13-Apr-2016 14:47:17 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

50202.0 step3

runTheMatrix-results/50202.0_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50/step3_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50.log
----- Begin Fatal Exception 13-Apr-2016 15:06:18 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

25202.0 step3

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25/step3_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25.log
----- Begin Fatal Exception 13-Apr-2016 15:09:22 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=PATPFJetMETcorrInputProducer label='patPFMetT1T2Corr'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'isMC'
 'type2ExtraCorrFactor'
 'type2ResidualCorrEtaMax'
 'type2ResidualCorrLabel'
 'type2ResidualCorrOffset'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

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

@cmsbuild
Copy link
Contributor

Pull request #14039 was updated. @cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please check and sign again.

@Martin-Grunewald
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@cvuosalo
Copy link
Contributor

+1

For #14039 31d1798

Adding fillDescriptions methods to two MET producer modules. There should be no change in monitored quantities.

#14040 is the 81X version of this PR.

The code changes are satisfactory, and Jenkins tests against baseline CMSSW_8_0_X_2016-04-14-2300 show no significant differences, as expected.

@davidlange6 davidlange6 merged commit 0ff0444 into cms-sw:CMSSW_8_0_X Apr 18, 2016
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

6 participants