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

Add two photon sc eta bits to nanoaod #123

Merged
merged 3 commits into from Mar 19, 2018

Conversation

AndrewLevin
Copy link

No description provided.

@gpetruc-bot
Copy link

@gpetruc-bot
Copy link

Please update PhysicsTools/NanoAOD/python/nanoDQM_cfi.py: take this patch or run prepareDQM.py -d -u nano_file_mc.root, and then if needed adjust the plot range using some human common sense.

Copy link

@gpetruc-bot gpetruc-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automatic test report for 334183

Code integration

Code checks not run for this PR (no source files modified)

Please update PhysicsTools/NanoAOD/python/nanoDQM_cfi.py: take this patch or run prepareDQM.py -d -u nano_file_mc.root, and then if needed adjust the plot range using some human common sense.

Tests

  • Long test data80X (10000 events): passed, no significant changes; dqm plots: all, diff
  • Long test data80Xhip (3000 events): passed, no significant changes; dqm plots: all, diff
  • Long test data94X (10000 events): passed, no significant changes; dqm plots: all, diff
  • Long test mc80X (10000 events): passed, no significant changes; dqm plots: all, diff
  • Long test mc94X (10000 events): passed, no significant changes; dqm plots: all, diff
  • Test mc_94X: passed
  • Test mc_80X: passed
  • Test mc_92X: passed
  • Test data_92X: passed
  • Test data_80X: passed

Disk size report

Sample kb/event ref kb/event diff
TTbar MC 94X 1.511 1.509 0.002 ( +0.1% )
TTbar MC 80X 1.559 1.556 0.003 ( +0.2% )
Data 94X 0.612 0.611 0.001 ( +0.2% )
Data 80X 0.578 0.576 0.002 ( +0.3% )
Data 80X, Mu Run2016E 0.508 0.507 0.001 ( +0.1% )

@gpetruc gpetruc merged commit 9b36db1 into cms-nanoAOD:master Mar 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants