Add PFMET with FixEE2017 recipe #230
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Replaced with a clean commit on the current master-102X, avoiding conflicts |
gpetruc-bot
added
test scheduled (bot)
test started (bot)
and removed
to be tested by bot
test scheduled (bot)
labels
|
Automatic test started, see https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/589531/builds |
gpetruc-bot
reviewed
Nov 15, 2018
There was a problem hiding this comment.
Automatic test report for 589531
- gitlab pipeline at https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/589531/builds
- outputs at https://cms-nanoaod-integration.web.cern.ch/integration/test_pr_230/
Code integration
Code checks passed for this PR
Tests
- Long test data101X (10000 events): passed, no significant changes; dqm plots: all, diff
- 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 data94X2016 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test data94Xv2 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc102X (9000 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
- Long test mc94X2016 (9000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc94Xv2 (9000 events): passed, no significant changes; dqm plots: all, diff
- Test mc_94Xv2: passed
- Test mc_102X: passed
- Test data_94X: passed
- Test data_101X: passed
Disk size report
| Sample | kb/event | ref kb/event | diff |
|---|---|---|---|
| TTbar MC 94Xv1 | 1.643 | 1.630 | 0.013 ( +0.8% ) |
| TTbar MC 94Xv2 | 1.658 | 1.646 | 0.012 ( +0.7% ) |
| TTbar MC 94X2016 | 1.505 | 1.505 | -0.000 ( -0.0% ) |
| TTbar MC 80X | 1.654 | 1.655 | -0.001 ( -0.0% ) |
| Data 94Xv1 | 0.708 | 0.695 | 0.013 ( +1.8% ) |
| Data 94Xv2 | 0.714 | 0.701 | 0.013 ( +1.8% ) |
| Data 94X2016 | 0.707 | 0.707 | 0.000 ( +0.0% ) |
| Data 80X | 0.604 | 0.604 | 0.000 ( +0.0% ) |
| Data 80X, Mu Run2016E | 0.601 | 0.601 | -0.000 ( -0.0% ) |
peruzzim
added
to be tested by bot
and removed
test ok (bot)
code checks ok (bot)
labels
gpetruc-bot
added
test scheduled (bot)
and removed
to be tested by bot
test scheduled (bot)
labels
|
Automatic test started, see https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/589719/builds |
gpetruc-bot
reviewed
Nov 15, 2018
There was a problem hiding this comment.
Automatic test report for 589719
- gitlab pipeline at https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/589719/builds
- outputs at https://cms-nanoaod-integration.web.cern.ch/integration/test_pr_230/
Code integration
Code checks passed for this PR
Tests
- Long test data101X (10000 events): passed, no significant changes; dqm plots: all, diff
- 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 data94X2016 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test data94Xv2 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc102X (9000 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
- Long test mc94X2016 (9000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc94Xv2 (9000 events): passed, no significant changes; dqm plots: all, diff
- Test mc_94Xv2: passed
- Test mc_102X: passed
- Test data_94X: passed
- Test data_101X: passed
Disk size report
| Sample | kb/event | ref kb/event | diff |
|---|---|---|---|
| TTbar MC 94Xv1 | 1.643 | 1.630 | 0.012 ( +0.8% ) |
| TTbar MC 94Xv2 | 1.658 | 1.646 | 0.012 ( +0.7% ) |
| TTbar MC 94X2016 | 1.504 | 1.505 | -0.001 ( -0.1% ) |
| TTbar MC 80X | 1.653 | 1.655 | -0.001 ( -0.1% ) |
| Data 94Xv1 | 0.708 | 0.695 | 0.013 ( +1.8% ) |
| Data 94Xv2 | 0.714 | 0.701 | 0.013 ( +1.8% ) |
| Data 94X2016 | 0.707 | 0.707 | 0.000 ( +0.0% ) |
| Data 80X | 0.604 | 0.604 | 0.000 ( +0.1% ) |
| Data 80X, Mu Run2016E | 0.601 | 0.601 | 0.000 ( +0.0% ) |
peruzzim
added
to be tested by bot
and removed
test ok (bot)
to be tested by bot
labels
gpetruc-bot
added
test scheduled (bot)
test started (bot)
and removed
to be tested by bot
test scheduled (bot)
labels
|
Automatic test started, see https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/590196/builds |
gpetruc-bot
reviewed
Nov 15, 2018
There was a problem hiding this comment.
Automatic test report for 590196
- gitlab pipeline at https://gitlab.cern.ch/cms-nanoAOD/nanoAOD-integration/pipelines/590196/builds
- outputs at https://cms-nanoaod-integration.web.cern.ch/integration/test_pr_230/
Code integration
Code checks passed for this PR
Tests
- Long test data101X (10000 events): passed, no significant changes; dqm plots: all, diff
- 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 data94X2016 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test data94Xv2 (10000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc102X (9000 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
- Long test mc94X2016 (9000 events): passed, no significant changes; dqm plots: all, diff
- Long test mc94Xv2 (9000 events): passed, no significant changes; dqm plots: all, diff
- Test mc_94Xv2: passed
- Test mc_102X: passed
- Test data_94X: passed
- Test data_101X: passed
Disk size report
| Sample | kb/event | ref kb/event | diff |
|---|---|---|---|
| TTbar MC 94Xv1 | 1.643 | 1.631 | 0.012 ( +0.7% ) |
| TTbar MC 94Xv2 | 1.658 | 1.646 | 0.011 ( +0.7% ) |
| TTbar MC 94X2016 | 1.504 | 1.504 | 0.000 ( +0.0% ) |
| TTbar MC 80X | 1.660 | 1.660 | -0.001 ( -0.0% ) |
| Data 94Xv1 | 0.708 | 0.695 | 0.013 ( +1.9% ) |
| Data 94Xv2 | 0.715 | 0.702 | 0.013 ( +1.8% ) |
| Data 94X2016 | 0.707 | 0.707 | -0.000 ( -0.0% ) |
| Data 80X | 0.615 | 0.616 | -0.000 ( -0.1% ) |
| Data 80X, Mu Run2016E | 0.603 | 0.603 | 0.000 ( +0.1% ) |
|
Tested with cms-sw#25173 |
Conflicts: PhysicsTools/NanoAOD/python/nanoDQM_cff.py
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Needs cms-sw#24876 + cms-sw#25129 backported to 94X (cms-sw#25035) and 102X (cms-sw#25173)