dqm for DoubleMu3_DCA_PFMET50_PFMHT60 #20584
Conversation
|
The code-checks are being triggered in jenkins. |
|
+code-checks |
|
A new Pull Request was created by @gkaratha for master. It involves the following packages: DQMOffline/Trigger @kmaeshima, @cmsbuild, @vanbesien, @vazzolini, @dmitrijus can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
+1 please read https://www.python.org/dev/peps/pep-0008/ in your spare time. |
cmsbuild
added
dqm-approved
fully-signed
and removed
dqm-pending
pending-signatures
labels
|
The tests are being triggered in jenkins. |
|
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 will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar (and backports should be raised in the release meeting by the corresponding L2) |
|
can you please quantify the increase in #bins ? |
|
Dear @mtosi , I did not increase the bins. They are the same as in DoubleMu3_DZ_PFMET50_PFMHT60. Or I did not understand the question? Thanks |
|
are you adding new histograms ?
…On Tue, Sep 19, 2017 at 6:20 PM, gkaratha ***@***.***> wrote:
Dear @mtosi <https://github.com/mtosi> ,
I did not increase the bins. They are the same as in
DoubleMu3_DZ_PFMET50_PFMHT60. Or I did not understand the question? Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#20584 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AEt58yKBOg9LkIlISYJIHhRWXOnBNpsuks5sj-nXgaJpZM4Pcg7g>
.
|
|
Yes for a new path |
|
can you quantify the #bins you are adding, please ?
…On Tue, Sep 19, 2017 at 6:49 PM, gkaratha ***@***.***> wrote:
Yes for a new path
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#20584 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AEt58-ct6gZDyjjeSEHcKuul7jed5dLXks5sj_CjgaJpZM4Pcg7g>
.
|
|
ok sure! if i am not mistaken 354 in total, for all different plots (meaning i dont count numerator/denominator) . But is less than in the doublemu3_dz_pfmet50_pfmht60. I created less plots now. Thanks |
|
also i can remove some plots that i think are redundant (like 2d for muon pt) |
|
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
|
The code-checks are being triggered in jenkins. |
|
+code-checks |
|
Pull request #20584 was updated. @kmaeshima, @cmsbuild, @vanbesien, @vazzolini, @dmitrijus can you please check and sign again. |
|
@cmsbuild, please test |
|
The tests are being triggered in jenkins. |
|
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
|
Comparison job queued. |
|
Comparison is ready Comparison Summary:
|
|
@dmitrijus @davidlange6 |
|
+1 |
cmsbuild
added
dqm-approved
fully-signed
and removed
dqm-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
Dear experts,
No new code developed. Only reuse what was already there for a modified path. Thanks