xml files containing updated summary plot limit + fix for cosmic [Backport combining #19447 and #19713]

[boudoul]

Greetings
xml files for the qTests for the DQM PixelPhase1 summary plots and fix for cosmic sequence

this is the backport of #19447 and #19713

I combined the two since #19713 is fixing #19447 - Those are already merged in 93X (and even part of 930_pre2)
@ckmackay FYI

[cmsbuild]

A new Pull Request was created by @boudoul (boudoul) for CMSSW_9_2_X.

It involves the following packages:

DQM/SiPixelPhase1Config
RecoTracker/Configuration

@perrotta, @vazzolini, @kmaeshima, @dmitrijus, @cmsbuild, @slava77, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @hdelanno, @makortel, @felicepantaleo, @GiacomoSguazzoni, @rovere, @VinInn, @mschrode, @fioriNTU, @gpetruc, @idebruyn, @ebrondol, @threus, @dgulhan this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[ckmackay]

Many thanks, I have unfortunately been away and not caught up with this. Cheers, Kate

…

________________________________ From: cmsbuild [notifications@github.com] Sent: 19 July 2017 15:43 To: cms-sw/cmssw Cc: Catherine Mackay; Mention Subject: Re: [cms-sw/cmssw] xml files containing updated summary plot limit + fix for cosmic [Backport combining #19447 and #19713] (#19821) A new Pull Request was created by @boudoul<https://github.com/boudoul> (boudoul) for CMSSW_9_2_X. It involves the following packages: DQM/SiPixelPhase1Config RecoTracker/Configuration @perrotta<https://github.com/perrotta>, @vazzolini<https://github.com/vazzolini>, @kmaeshima<https://github.com/kmaeshima>, @dmitrijus<https://github.com/dmitrijus>, @cmsbuild<https://github.com/cmsbuild>, @slava77<https://github.com/slava77>, @vanbesien<https://github.com/vanbesien>, @davidlange6<https://github.com/davidlange6> can you please review it and eventually sign? Thanks. @ghellwig<https://github.com/ghellwig>, @hdelanno<https://github.com/hdelanno>, @makortel<https://github.com/makortel>, @felicepantaleo<https://github.com/felicepantaleo>, @GiacomoSguazzoni<https://github.com/giacomosguazzoni>, @rovere<https://github.com/rovere>, @VinInn<https://github.com/vininn>, @mschrode<https://github.com/mschrode>, @fioriNTU<https://github.com/fiorintu>, @gpetruc<https://github.com/gpetruc>, @idebruyn<https://github.com/idebruyn>, @ebrondol<https://github.com/ebrondol>, @threus<https://github.com/threus>, @dgulhan<https://github.com/dgulhan> this is something you requested to watch as well. @davidlange6<https://github.com/davidlange6> you are the release manager for this. cms-bot commands are listed here<http://cms-sw.github.io/cms-bot-cmssw-cmds.html> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#19821 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AKXhhMk9qz8XZK_WBMRBV3HUQ1Xn42nRks5sPgfqgaJpZM4Ocv4h>.

[perrotta]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/21616/console Started: 2017/07/19 20:22

[cmsbuild]

+1
Tested at: 13ebe3b
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19821/21616/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
ae7effd
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19821/21616/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19821/21616/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 22
• DQMHistoTests: Total histograms compared: 1756428
• DQMHistoTests: Total failures: 15042
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1741220
• DQMHistoTests: Total skipped: 166
• DQMHistoTests: Total Missing objects: 0
• Checked 90 log files, 14 edm output root files, 22 DQM output files

[slava77]

+1

for #19821 13ebe3b

• reco changes are only in RecoTracker/Configuration/python/RecoTrackerP5_cff.py , where a module siPixelClusterShapeCache is added to tracksP5. This is a pre-emptive bugfix to 92X PR backport of 19618 #19629 (backport of Enhance DQM for Pixel cluster on tracks #19618 which introduced problems in cosmics workflows)
• jenkins tests pass
  • there are no changes in RECO [as expected]
  • there are some changes in "PixelPhase1/Run summary/Summary" and a related summary plot in /EventInfo folder

[dmitrijus]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_3_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[davidlange6]

+1