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

EGM IDs for CMSSW 74X #9876

Closed
wants to merge 41 commits into from
Closed

EGM IDs for CMSSW 74X #9876

wants to merge 41 commits into from

Conversation

lgray
Copy link
Contributor

@lgray lgray commented Jun 24, 2015

Contains a backport of #9797, without the ExpressionParser.

Also implements VID cutbased and MVA-based electron and photon IDs for PHYS 14 and Spring15.

This will require some new entries into the appropriate cms-data repositories.
A forward port to CMSSW_7_5_X and CMSSW_7_6_X will come soon.

@ikrav @cmkuo

lgray and others added 30 commits June 17, 2015 17:41
@lgray
Copy link
Contributor Author

lgray commented Jul 1, 2015

Externals delayed because of development on the later, and yes there will
be a backport.
Expect to be done Friday.

-L

On Wed, Jul 1, 2015 at 9:52 PM, Slava Krutelyov notifications@github.com
wrote:

@cmsbuild https://github.com/cmsbuild please test

@lgray https://github.com/lgray

  • What's the status of moving the externals?
  • is the miniAOD-from-miniAOD part going to show up here as well?


Reply to this email directly or view it on GitHub
#9876 (comment).

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 1, 2015

-1
Tested at: 42a5144
When I ran the RelVals I found an error in the following worklfows:
50202.0 step2

runTheMatrix-results/50202.0_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50+MINIAODMCUP1550/step2_TTbar_13+TTbar_13+DIGIUP15_PU50+RECOUP15_PU50+HARVESTUP15_PU50+MINIAODMCUP1550.log
----- Begin Fatal Exception 01-Jul-2015 22:40:25 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'digitisation_step'
   [2] Calling event method for module MixingModule/'mix'
Exception Message:
RootInputFileSequence::readOneRandom(): no input files specified for secondary input source.
----- End Fatal Exception -------------------------------------------------

25202.0 step3

runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25+MINIAODMCUP15/step3_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25+MINIAODMCUP15.log
----- Begin Fatal Exception 01-Jul-2015 22:48:07 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'validation_step'
   [2] Calling event method for module MixingModule/'mix'
Exception Message:
RootInputFileSequence::readOneSpecified(): no input files specified for secondary input source.
----- End Fatal Exception -------------------------------------------------

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

@slava77
Copy link
Contributor

slava77 commented Jul 2, 2015

@lgray please also resolve the event content issue that appears when running reco+miniaod in one application slava77#71
The {egmGsfElectronIDs} from the RECO part should be explicitly specified in the RECO*EventContent
http://cmslxr.fnal.gov/lxr/source/RecoEgamma/Configuration/python/RecoEgamma_EventContent_cff.py?v=CMSSW_7_4_6#0016
Thank you.

@lgray
Copy link
Contributor Author

lgray commented Jul 2, 2015

Sure, that's no problem.

On Thu, Jul 2, 2015 at 7:41 PM, Slava Krutelyov notifications@github.com
wrote:

@lgray https://github.com/lgray please also resolve the event content
issue that appears when running reco+miniaod in one application slava77#71
slava77#71
The {egmGsfElectronIDs} from the RECO part should be explicitly specified
in the RECO*EventContent

http://cmslxr.fnal.gov/lxr/source/RecoEgamma/Configuration/python/RecoEgamma_EventContent_cff.py?v=CMSSW_7_4_6#0016
Thank you.


Reply to this email directly or view it on GitHub
#9876 (comment).

@lgray lgray closed this Jul 4, 2015
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.

6 participants