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

Fix nanoAOD default config for 2018 without era (94X version, to check it doesn't change anything in 94X) #179

Merged

Conversation

gpetruc
Copy link

@gpetruc gpetruc commented Jul 5, 2018

94X version of cms-sw#23749 for testing

@gpetruc gpetruc changed the title Fix nano aod 2018 no era 94 x Fix nanoAOD default config for 2018 without era (94X version to check it doesn Jul 5, 2018
@gpetruc gpetruc changed the title Fix nanoAOD default config for 2018 without era (94X version to check it doesn Fix nanoAOD default config for 2018 without era (94X version, to check it doesn't change anything in 94X) Jul 5, 2018
@gpetruc gpetruc added the HOLD label Jul 5, 2018
@gpetruc-bot
Copy link

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 436596

Code integration

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

Tests

  • Test mc_94Xv2: passed
  • Test mc_94X: passed
  • Test mc_80X: passed
  • Test data_94Xv2: passed
  • Test data_94X: passed
  • Test data_80X: passed

@gpetruc-bot
Copy link

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 436809

Code integration

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

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 data94Xv2 (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
  • Long test mc94Xv2 (9000 events): passed, no significant changes; dqm plots: all, diff
  • Test mc_94Xv2: passed
  • Test mc_94X: passed
  • Test mc_80X: passed
  • Test data_94Xv2: passed
  • Test data_94X: passed
  • Test data_80X: passed

Disk size report

Sample kb/event ref kb/event diff
TTbar MC 94Xv1 1.572 1.572 -0.000 ( -0.0% )
TTbar MC 94Xv2 1.605 1.604 0.001 ( +0.1% )
TTbar MC 80X 1.610 1.609 0.001 ( +0.0% )
Data 94Xv1 0.631 0.631 -0.000 ( -0.0% )
Data 80X 0.580 0.580 0.000 ( +0.0% )
Data 80X, Mu Run2016E 0.577 0.577 0.000 ( +0.0% )

@gpetruc gpetruc changed the base branch from master to old_master_946p1 July 17, 2018 11:06
@gpetruc gpetruc removed the HOLD label Jul 17, 2018
@gpetruc
Copy link
Author

gpetruc commented Jul 17, 2018

will merge even if it has no effect, to help keep master and 94X more similar and rebases easier

@gpetruc gpetruc merged commit 77a6ca4 into cms-nanoAOD:old_master_946p1 Jul 17, 2018
@gpetruc gpetruc deleted the fix_nanoAOD_2018_noEra_94X branch March 8, 2021 09:09
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.

2 participants