Updated Z->ee skim for Run2

[emanueledimarco]

This is an updated skim for Z->ee for Run2. The T&P pairs have been removed to keep the efficiency on DoubleElectron sufficiently low to have the skim run centrally.

The cuts are:

• pT > 20 / 10 GeV
• electron ID veto (including PF isolation)
• 40 < mass < 140 GeV

the selection has been synchronised with the AlcaElectron stream.

[cmsbuild]

A new Pull Request was created by @emanueledimarco (Emanuele Di Marco) for CMSSW_7_5_X.

Updated Z->ee skim for Run2

It involves the following packages:

DPGAnalysis/Skims

@cmsbuild, @srimanob, @boudoul, @franzoni 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[franzoni]

@alkaloge
have you been able to test this ?

[alkaloge]

cmsDriver crashes for me (7_5_0_pre3)
cmsDriver.py skims -s SKIM:ZElectron --data --dasquery="file dataset=/SingleElectron/CMSSW_7_4_0-GR_R_74_V10A_RelVal_electron2012A-v1/RECO" -n 100 --conditions auto:com10 --python_filename=ZElectron_SKIM.py --processName=SKIMZElectron

complains for "NameError: name 'tagGsfSeq' is not defined" (basically you need to change accordingly the Skims_DPG_cff.py file

@emanueledimarco can you check ? Also please remove DPGAnalysis/Skims/python/test_ZeeSkim.py from this dir

[cmsbuild]

Pull request #9548 was updated. @cmsbuild, @srimanob, @boudoul, @franzoni can you please check and sign again.

[emanueledimarco]

Sorry, I didn't know about the summary cff file. Now I have updated it and tested on DoubleElectron [1]
@shervin86 please also follow this.

[1] cmsDriver.py skims -s SKIM:ZElectron --data --dasquery="file dataset=/DoubleElectron/CMSSW_7_4_3-GR_R_74_V12A_RelVal_zEl2012D-v5/RECO" -n 100 --conditions auto:com10 --python_filename=ZElectron_SKIM.py --processName=SKIMZElectron

[alkaloge]

tested - looks good to go

[franzoni]

thanks @alkaloge

[franzoni]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: a44fde2
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9548/5462/summary.html

[franzoni]

+1

[cmsbuild]

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

[davidlange6]

@emanueledimarco - could you remove the new printout?

[emanueledimarco]

@davidlange6 done here and in the 74X back port #9549

[cmsbuild]

Pull request #9548 was updated. @cmsbuild, @srimanob, @boudoul, @franzoni can you please check and sign again.

[franzoni]

please test

[franzoni]

+1
sicne the only change is print