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

NanoAOD update #21095

Merged
merged 81 commits into from Nov 14, 2017
Merged

NanoAOD update #21095

merged 81 commits into from Nov 14, 2017

Conversation

arizzi
Copy link
Contributor

@arizzi arizzi commented Oct 31, 2017

  • Add several variables in NanoAOD.
  • Read from miniaod some of the new variables computed in pre3 (e.g. for jet substructures)
  • Move cmsDriver input files to pre3
  • Includes DQM for NanoAOD

gouskos and others added 30 commits October 2, 2017 21:07
Added non-PF track, ecal and hcal isolation within a cone of 0.3
- cmsDriver.py mc92X_dq -s NANO,DQM:@nanoAODDQM --mc --eventcontent NANOAODSIM,DQM --datatier NANO,DQMIO --filein /store/mc/RunIISummer17MiniAOD/TT_TuneCUETP8M2T4_13TeV-powheg-pythia8/MINIAODSIM/92X_upgrade2017_realistic_v10_ext1-v2/50000/024FE157-9899-E711-B553-001E67E71D03.root --conditions auto:phase1_2017_realistic -n 10000 --nThreads 4 --era Run2_2017,run2_nanoAOD_92X
- cmsDriver.py mc92X_dq --conditions auto:phase1_2017_realistic -s HARVESTING:@nanoAODDQM --era Run2_2017 --filein file:mc92X_dq_NANO_DQM_inDQM.root --scenario pp --filetype DQM --geometry DB:Extended --mc  -n -1
@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/PR-21095/1986

@cmsbuild
Copy link
Contributor

Pull request #21095 was updated. @perrotta, @prebello, @monttj, @vazzolini, @kmaeshima, @kpedro88, @fabozzi, @cmsbuild, @jfernan2, @slava77, @GurpreetSinghChahal, @vanbesien, @dmitrijus can you please check and sign again.

@gpetruc
Copy link
Contributor

gpetruc commented Nov 14, 2017 via email

@kpedro88
Copy link
Contributor

@mdhildreth suggests both new packages should go in the analysis category (following the example of PAT)

@davidlange6
Copy link
Contributor

packages are added now

@davidlange6
Copy link
Contributor

merge

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