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

miniAOD part4 (7.2.X, only PatAlgos and cmsDriver) #4016

Merged
merged 5 commits into from Jun 10, 2014

Conversation

gpetruc
Copy link
Contributor

@gpetruc gpetruc commented May 27, 2014

Propagate a few committs that we had only in the 70X part3 pull request #4008 and not in the merged 72X PR (#3946) and avoid another ak4-ak5 issue in mc flavour for met uncertainties (which does not apply to 70X)

Will backport to 71X when #3875 wil be merged (it's the same branch as the already merged #3946, and weeks old, so I hope it will go in soon)

Tested with

# after runTheMatrix.py -l 4.75,25202.0,25401.0
cmsDriver.py miniAOD_data-prod -s PAT -n 100 --data      --filein file:$PWD/4.75_RunJet2012D+RunJet2012D+HLTD+RECODreHLT+HARVESTDreHLT/step3.root     --eventcontent MINIAOD    --runUnscheduled --conditions auto:com10_GRun --scenario pp --no_exec
cmsDriver.py miniAOD-prod      -s PAT -n 100 --mc        --filein file:$PWD/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVEST/step3.root  --eventcontent MINIAODSIM --runUnscheduled --conditions auto:run2_mc    --magField 38T_PostLS1 --customise SLHCUpgradeSimulations/Configuration/postLS1Customs.customisePostLS1 --no_exec
cmsDriver.py miniAOD-fsim-prod -s PAT -n 100 --mc --fast --filein file:$PWD/25401.0_TTbar_13_AVE20+TTbarFSPU13AVE20+HARVESTFS/step1.root              --eventcontent MINIAODSIM --runUnscheduled --conditions auto:run2_mc    --magField 38T_PostLS1 --customise SLHCUpgradeSimulations/Configuration/postLS1Customs.customisePostLS1 --no_exec

@arizzi

…esn't get added to the jets used for MET systematics (fixes regression after bugfix to to the PAT UserData Merger)
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @gpetruc (Giovanni Petrucciani) for CMSSW_7_2_X.

miniAOD part4 (7.2.X, only PatAlgos and cmsDriver)

It involves the following packages:

Configuration/Applications
PhysicsTools/PatAlgos

@nclopezo, @vlimant, @monttj, @cmsbuild, @franzoni, @vadler, @Degano, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @TaiSakuma, @nhanvtran this is something you requested to watch as well.
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.

@cmsbuild
Copy link
Contributor

-1
When I ran the RelVals I found an error in the following worklfows:
1000.0 step3

runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step3_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log
----- Begin Fatal Exception 27-May-2014 14:16:50 CEST-----------------------
An exception of category 'FileReadError' occurred while
   [0] Processing run: 165121 lumi: 62 event: 23679717
   [1] Running path 'pathlogerror'
   [2] Calling event method for module L1GlobalTriggerEvmRawToDigi/'gtEvmDigis'
   [3] Reading branch FEDRawDataCollection_source__HLT.
   Additional Info:
      [a] Fatal Root Error: @SUB=TBranchElement::GetBasket
File: root://eoscms//eos/cms/store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root at byte:7839505, branch:FEDRawDataCollection_source__HLT.obj, entry:52, badread=1, nerrors=1, basketnumber=52
----- End Fatal Exception -------------------------------------------------

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

@gpetruc
Copy link
Contributor Author

gpetruc commented May 27, 2014

this is an I/O failure that has nothing to do with this PR.

On Tue, May 27, 2014 at 2:25 PM, cmsbuild notifications@github.com wrote:

-1
When I ran the RelVals I found an error in the following worklfows:
1000.0 step3

runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step3_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log

----- Begin Fatal Exception 27-May-2014 14:16:50 CEST-----------------------

An exception of category 'FileReadError' occurred while

[0] Processing run: 165121 lumi: 62 event: 23679717

[1] Running path 'pathlogerror'

[2] Calling event method for module L1GlobalTriggerEvmRawToDigi/'gtEvmDigis'

[3] Reading branch FEDRawDataCollection_source__HLT.

Additional Info:

  [a] Fatal Root Error: @SUB=TBranchElement::GetBasket

File: root://eoscms//eos/cms/store/data/Run2011A/MinimumBias/RAW/v1/000/165/121/0699429A-B37F-E011-A57A-0019B9F72D71.root at byte:7839505, branch:FEDRawDataCollection_source__HLT.obj, entry:52, badread=1, nerrors=1, basketnumber=52

----- End Fatal Exception -------------------------------------------------

you can see the results of the tests here:

https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4016/22/summary.html


Reply to this email directly or view it on GitHubhttps://github.com//pull/4016#issuecomment-44268322
.

@vadler
Copy link

vadler commented May 27, 2014

+1

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@arizzi
Copy link
Contributor

arizzi commented Jun 10, 2014

what's the matter with this PR? why it needs more than two weeks to merge even on a just open release cycle, with only a few commits? ... AT signed two weeks ago...

@davidlange6
Copy link
Contributor

sorry - missed it . pushing it now

davidlange6 added a commit that referenced this pull request Jun 10, 2014
miniAOD part4 (7.2.X, only PatAlgos and cmsDriver)
@davidlange6 davidlange6 merged commit 0a7c5a4 into cms-sw:CMSSW_7_2_X Jun 10, 2014
@arizzi
Copy link
Contributor

arizzi commented Jun 10, 2014

thanks!

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

5 participants