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

CMSSW_7_5_X DQM updates for AlphaT HLT paths (v2) #10878

Merged

Conversation

MarkBaber
Copy link
Contributor

Addition of DQM monitoring to new AlphaT path, supersedes #10704

Addition of DQM monitoring for the following HLT paths:

  • HLT_PFHT200_PFAlphaT0p51_v
  • HLT_PFHT200_DiPFJetAve90_PFAlphaT0p57_v
  • HLT_PFHT250_DiPFJetAve90_PFAlphaT0p55_v
  • HLT_PFHT300_DiPFJetAve90_PFAlphaT0p53_v
  • HLT_PFHT350_DiPFJetAve90_PFAlphaT0p52_v
  • HLT_PFHT400_DiPFJetAve90_PFAlphaT0p51_v

Remove number of monitoring histograms

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @MarkBaber (Mark Baber) for CMSSW_7_5_X.

CMSSW_7_5_X DQM updates for AlphaT HLT paths (v2)

It involves the following packages:

HLTriggerOffline/SUSYBSM

@cmsbuild, @danduggan, @deguio can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@danduggan
Copy link
Contributor

please test
@MarkBaber, yes please do close your other PRs and we'll continue with these.

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

@danduggan
Copy link
Contributor

+1
@MarkBaber, the difference in bins is much more manageable now. for the direct commenting out of code, it's not too pretty and I'd avoid it, but I'll let @davidlange6 comment on integrating as is.

@cmsbuild
Copy link
Contributor

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

@MarkBaber
Copy link
Contributor Author

#10877 has been merged, can we also merge this?

@davidlange6
Copy link
Contributor

+1

cmsbuild added a commit that referenced this pull request Sep 1, 2015
CMSSW_7_5_X DQM updates for AlphaT HLT paths (v2)
@cmsbuild cmsbuild merged commit f5bc1a6 into cms-sw:CMSSW_7_5_X Sep 1, 2015
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.

4 participants