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 ID Updates from recipes (76X) #11738
EGM ID Updates from recipes (76X) #11738
Conversation
…VA V1 for both 50ns and 25ns of Spring15
…ial, a linear scaling was used for neutral hadron isolation cut for the endcap. Fixed now.
A new Pull Request was created by @lgray (Lindsey Gray) for CMSSW_7_6_X. EGM ID Updates from recipes (76X) It involves the following packages: PhysicsTools/PatAlgos @cmsbuild, @cvuosalo, @vadler, @monttj, @slava77 can you please review it and eventually sign? Thanks. |
@lgray here and in your other two PRs I'm not going to trigger tests, assuming you will detect when the IB with the externals show up |
@slava77 Yes, I'll take care of starting the tests. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 |
Errors seem to be with runTheMatrix.py itself:
and then it quits... Will wait for the IBs to be fixed before proceeding. |
please test |
The tests are being triggered in jenkins. |
-1 runTheMatrix-results/4.53_RunPhoton2012B+RunPhoton2012B+HLTD+RECODreHLT+HARVESTDreHLT/step3_RunPhoton2012B+RunPhoton2012B+HLTD+RECODreHLT+HARVESTDreHLT.log 135.4 step3 runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step3_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log 1306.0 step3 runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15/step3_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15.log 1330.0 step3 runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15/step3_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15.log 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 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 you can see the results of the tests here: |
@cmsbuild please test |
The tests are being triggered in jenkins. |
theBeamSpot->position()); | ||
double vertexFitProbability = -1.; | ||
if(!conv_ref.isNull()) { | ||
const reco::Vertex &vtx = conv_ref.get()->conversionVertex(); if (vtx.isValid()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new line in front of the "if" would be nice
+1
|
EGM ID Updates from recipes (76X)
This PR is brings CMSSW in line with the latest EGM recipes.
New IDs:
Bugfix:
Externals PR is here: cms-data/RecoEgamma-ElectronIdentification#6
Please wait until this is in an IB before testing.
@ikrav