Fixes(need changes outside the repository as well), to enable success…

ECALELFS / ECALELF

Code for electron calibration in CMS

http://ecalelfs.github.io/ECALELF/index.html

2 stars 22 forks source link

Fixes(need changes outside the repository as well), to enable success… #111

Open rchatter opened 7 years ago

rchatter commented 7 years ago

DO NOT MERGE WITH MASTER: Fixes(need changes outside the repository as well), to enable successful running of Recal->nTuple production. To successfully Run: cmsRun python/alcaSkimming.py tagFile=config/reRecoTags 80X_mcRun2_asymptotic_2016_TrancheIV_v7.py type=ALCARERECO files=file:Uncal_To_Recal_001.root doTree=1 doTreeOnly=1 isPrivate=1 MC=1 This branch is needed(To be confirmed if the fixes break something else). ADDITIONALLY need to: git cms-addpkg PhysicsTools/PatAlgos cp ~rchatter/public/PATElectronProducer.cc /PhysicsTools/PatAlgos/plugins/ scram b -j32

previsualconsent commented 7 years ago

You can handle the outside repository changes by doing a git cms-merge-topic

rchatter commented 7 years ago

thanks, will keep in mind next time.

On Mon, Feb 27, 2017 at 4:16 PM, Peter Hansen notifications@github.com wrote:

You can handle the outside repository changes by doing a git cms-merge-topic

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ECALELFS/ECALELF/pull/111#issuecomment-282874387, or mute the thread https://github.com/notifications/unsubscribe-auth/AFxLxCSattasg2tPZVsLoyjkzbDXHuciks5rg0tAgaJpZM4MNp29 .

Rajdeep.Mohan.Chatterjee Post Doctoral Fellow School of Physics and Astronomy University of Minnesota 115 Union Street Street SE Minneapolis MN 55455 USA

shervin86 commented 7 years ago

I've seen the PR. I won't have time till wednesday to test it. I don't fully understand the problem from the commits here. But if it works, don't hesitate and use it.