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

SUSY HLT DQM updates for trimuon and dimu+MET paths, 81X #14582

Merged
merged 3 commits into from May 25, 2016

Conversation

olivito
Copy link
Contributor

@olivito olivito commented May 19, 2016

This PR adds HLT DQM monitoring for the SUSY paths:
HLT_TripleMu_5_3_3
HLT_DoubleMu3_PFMET50

The code for the first is provided by Higgs PAG since they were already set up to monitor trimuon paths. For the second, existing susy muon+MET code is used.

This is the 81X version of the following PR in 80X:
#14581

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @olivito (Dominick Olivito) for CMSSW_8_1_X.

It involves the following packages:

HLTriggerOffline/Higgs
HLTriggerOffline/SUSYBSM

@cmsbuild, @dmitrijus, @vanbesien, @deguio, @davidlange6 can you please review it and eventually sign? Thanks.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@dmitrijus
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 24, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/13158/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@dmitrijus
Copy link
Contributor

Please check histograms in https://cmssdt.cern.ch/SDT/jenkins-artifacts/baseLineComparisons/CMSSW_8_1_X_2016-05-23-2300+14582/14122/alternative-comparisons/25202.0-result.pdf.gz

If those changes can be explained - I will +1, I see no issues with code itself.

@dmitrijus
Copy link
Contributor

+1

The difference in histograms are caused by this PR #14604

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit b13d0f7 into cms-sw:CMSSW_8_1_X May 25, 2016
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

4 participants