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

residual plots for DQM #17705

Merged
merged 6 commits into from Mar 17, 2017
Merged

Conversation

cmsbuild
Copy link
Contributor

@cmsbuild cmsbuild commented Mar 2, 2017

Mean and RMS of the residuals of the tracks have been added in the DQM. Details can be found in this presentation:
https://indico.cern.ch/event/536893/contributions/2389697/attachments/1388650/2114441/AnielloSpiezia_141216.pdf
Automatically ported from CMSSW_9_0_X #17645 (original by @aspiezia).
Please wait for a new IB (12 to 24H) before requesting to test this PR.

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 2, 2017

A new Pull Request was created by @cmsbuild for master.

It involves the following packages:

DQM/SiStripMonitorClient
DQM/TrackerMonitorTrack

@cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@hdelanno, @makortel, @fioriNTU, @idebruyn, @threus, @venturia this is something you requested to watch as well.
@Muzaffar, @davidlange6, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

@dmitrijus
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 7, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/18199/console Started: 2017/03/07 11:43

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 7, 2017

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @Muzaffar, @davidlange6, @smuzaffar

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 7, 2017

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 7, 2017

Comparison job queued.

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 7, 2017

@fioriNTU
Copy link
Contributor

Is this something else related to broken IB?

@davidlange6
Copy link
Contributor

davidlange6 commented Mar 16, 2017 via email

@davidlange6
Copy link
Contributor

davidlange6 commented Mar 16, 2017 via email

@fioriNTU
Copy link
Contributor

fioriNTU commented Mar 16, 2017 via email

@aspiezia
Copy link
Contributor

@fioriNTU, @davidlange6,
I am having a look.

@boudoul
Copy link
Contributor

boudoul commented Mar 16, 2017

suspiciously the same error has been seen in another PR test :
#17942 (comment)

@boudoul
Copy link
Contributor

boudoul commented Mar 16, 2017

or even this one #17832 (comment)
which was cured with a fresh test - Trackingmonitor became unstable ?

@davidlange6
Copy link
Contributor

davidlange6 commented Mar 16, 2017 via email

@cmsbuild
Copy link
Contributor Author

cmsbuild commented Mar 16, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/18498/console Started: 2017/03/16 17:32

@cmsbuild
Copy link
Contributor Author

@cmsbuild
Copy link
Contributor Author

Comparison job queued.

@cmsbuild
Copy link
Contributor Author

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 7b8177d into cms-sw:master Mar 17, 2017
@aspiezia
Copy link
Contributor

Hi,
is it not clear to me if it was a problem of runTheMatrix or if I need to further investigate.
Aniello

@dmitrijus
Copy link
Contributor

@aspiezia The problem was unrelated to this PR.

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

6 participants