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
Pixel Phase 1 summary maps [90X] #18035
Conversation
A new Pull Request was created by @leggat for CMSSW_9_0_X. It involves the following packages: DQM/SiPixelPhase1Config @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
For the record, this is a backport of the master (already merged) PR : #17880 |
+1 |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
Hey, |
@leggat please take the @vanbesien suggestion, we need this online |
Comparison job queued. |
Pull request #18035 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again. |
Hi, if this is a backport from the master branch, please make sure it has the same commits as the PR to that master branch. This PR contains 50 lines less than the one in master. |
Hi @dmitrijus - in fact this PR is more up-to-date than the merged one because of the bug fix introduced. I tried to make a new PR for 91X (#18146) to bring the merged code in line with this one, but it appears that 91X is now closed. How should I go about updating the already merged code? |
hi @leggat , 91X is not closed, it is just that it is the devlopment release and therefore you should make a PR to the default developement release called 'master ' which is the default queue when making a PR (and not choose 91X ) |
Ah sorry. Please find the new PR at #18148 |
+1 |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
Including the new summary maps for the phase 1 pixel detector in the 90X release.