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

Updates to DT, PF hadron, HCAL, PPS and pixel calibrations for 2018 UL [10_6_X] #27698

Conversation

christopheralanwest
Copy link
Contributor

PR description:

This is a backport of PR #27683. With this PR, 2018 collision GTs are up-to-date with the corresponding 11_0_X GTs:

https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_dataRun2_2017_2018_Candidate_2019_08_03_03_51_45/106X_dataRun2_2017_2018_Candidate_2019_08_05_17_19_13
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_dataRun2_relval_Candidate_2019_08_03_03_55_32/106X_dataRun2_relval_Candidate_2019_08_05_17_36_16
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_dataRun2_PromptLike_HEfail_Candidate_2019_08_02_16_55_52/106X_dataRun2_PromptLike_HEfail_Candidate_2019_08_05_17_40_11
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_dataRun2_PromptLike_Candidate_2019_08_02_16_53_14/106X_dataRun2_PromptLike_Candidate_2019_08_05_17_44_43
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_dataRun2_PromptLike_HI_Candidate_2019_08_02_16_55_18/106X_dataRun2_PromptLike_HI_Candidate_2019_08_05_17_49_31
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_upgrade2018_realistic_Candidate_2019_08_03_11_37_17/106X_upgrade2018_realistic_Candidate_2019_08_05_17_57_55
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/110X_upgrade2018_realistic_HEfail_Candidate_2019_08_03_11_39_50/106X_upgrade2018_realistic_HEfail_Candidate_2019_08_05_18_03_46

The GT diffs with respect to the current 10_6_X autoCond are:

https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_dataRun2_2017_2018_Candidate_2019_08_05_17_19_13/106X_dataRun2_v17
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_dataRun2_relval_Candidate_2019_08_05_17_36_16/106X_dataRun2_relval_v16
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_dataRun2_PromptLike_HEfail_Candidate_2019_08_05_17_40_11/106X_dataRun2_PromptLike_HEfail_v9
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_dataRun2_PromptLike_Candidate_2019_08_05_17_44_43/106X_dataRun2_PromptLike_v10
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_dataRun2_PromptLike_HI_Candidate_2019_08_05_17_49_31/106X_dataRun2_PromptLike_HI_v10
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_upgrade2018_realistic_Candidate_2019_08_05_17_57_55/106X_upgrade2018_realistic_v6
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/106X_upgrade2018_realistic_HEfail_Candidate_2019_08_05_18_03_46/106X_upgrade2018_realistic_HEfail_v6

This PR should be merged after #27605 so that clean PR comparison tests can be run on this PR. The merge conflict with #27605 will be resolved by preferring the changes in this PR. The backport of PR #27644 will also conflict this PR. Once either this PR or the backport of #27644 are merged, the GT candidates in the other PR will be converted to versioned GTs.

PR validation:

Validation as in PR #27683.

if this PR is a backport please specify the original PR:

Backport of #27683

@christopheralanwest
Copy link
Contributor Author

backport #27683

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2019

A new Pull Request was created by @christopheralanwest for CMSSW_10_6_X.

It involves the following packages:

Configuration/AlCa

@christopheralanwest, @tocheng, @cmsbuild, @franzoni, @tlampen, @pohsun can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @mmusich, @tocheng this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

@christopheralanwest
Copy link
Contributor Author

please test

@christopheralanwest christopheralanwest changed the title Updates to DT, PF hadron, HCAL, PPS and pixel calibrations for 2018 UL Updates to DT, PF hadron, HCAL, PPS and pixel calibrations for 2018 UL [10_6_X] Aug 5, 2019
@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2019

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/1842/console Started: 2019/08/05 22:38

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2019

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2019

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2019

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-35ca14/1842/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 11513 differences found in the comparisons
  • DQMHistoTests: Total files compared: 33
  • DQMHistoTests: Total histograms compared: 3208248
  • DQMHistoTests: Total failures: 11451
  • DQMHistoTests: Total nulls: 15
  • DQMHistoTests: Total successes: 3196448
  • DQMHistoTests: Total skipped: 334
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -0.051 KiB( 32 files compared)
  • DQMHistoSizes: changed ( 4.53 ): -0.016 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 136.731 ): -0.008 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 136.85 ): -0.031 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 1000.0 ): 0.004 KiB JetMET/SUSYDQM
  • Checked 137 log files, 14 edm output root files, 33 DQM output files

@christopheralanwest
Copy link
Contributor Author

The number of differences in the PR comparison tests for pre-2017 data workflows are the same as in PR #27605 so most likely this PR only affects 2018 data and MC.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2019

Pull request #27698 was updated. @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @tlampen, @pohsun can you please check and sign again.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-35ca14/1860/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 11611 differences found in the comparisons
  • DQMHistoTests: Total files compared: 33
  • DQMHistoTests: Total histograms compared: 3208248
  • DQMHistoTests: Total failures: 11469
  • DQMHistoTests: Total nulls: 15
  • DQMHistoTests: Total successes: 3196430
  • DQMHistoTests: Total skipped: 334
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 3206.914 KiB( 32 files compared)
  • DQMHistoSizes: changed ( 10224.0,... ): 132.521 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 140.56,... ): 166.837 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 4.22,... ): 60.878 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 25202.0,... ): 132.131 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 136.731 ): -0.008 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 29034.0,... ): 133.693 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 1000.0 ): 0.004 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 136.85 ): -0.031 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 7.3 ): 60.995 KiB Tracking/TrackParameters
  • DQMHistoSizes: changed ( 4.53 ): -0.016 KiB JetMET/SUSYDQM
  • DQMHistoSizes: changed ( 140.53 ): ...
  • Checked 137 log files, 14 edm output root files, 33 DQM output files

@christopheralanwest
Copy link
Contributor Author

please test

The tests are being run on the new IB to cross-check that there are no changes to pre-2018 workflows.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/1865/console Started: 2019/08/07 14:49

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 7, 2019

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-35ca14/1865/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 6040 differences found in the comparisons
  • DQMHistoTests: Total files compared: 33
  • DQMHistoTests: Total histograms compared: 3211049
  • DQMHistoTests: Total failures: 10793
  • DQMHistoTests: Total nulls: 7
  • DQMHistoTests: Total successes: 3199915
  • DQMHistoTests: Total skipped: 334
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -0.031 KiB( 32 files compared)
  • DQMHistoSizes: changed ( 136.85 ): -0.031 KiB JetMET/SUSYDQM
  • Checked 137 log files, 14 edm output root files, 33 DQM output files

@christopheralanwest
Copy link
Contributor Author

Changes are only seen in 2018 workflows: 136.85, 10824.0 and 250202.181. I would like to turn the candidate GTs into versioned GT but I will wait until the HCAL negative energy filter issue is resolved before deciding on the GT version numbering. auto:run2_data may change to 106X_dataRun2_v19 or 106X_dataRun2_v20 in the upcoming commit but I don't know which numbering would be most logical yet.

@kpedro88
Copy link
Contributor

@christopheralanwest just confirming that this can be closed because it's included in #27816

@christopheralanwest
Copy link
Contributor Author

Closing this PR because the relevant conditions are included in #27816.

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

3 participants