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

HLT integrations on top of 5198 #5202

Merged
merged 11 commits into from Sep 8, 2014

Conversation

perrotta
Copy link
Contributor

@perrotta perrotta commented Sep 6, 2014

It integrates in the run2 HLT menu the following tickets:

  • CMSHLT-87 (NoBPTX paths for EXO) added to the GRun menu
  • Electron Tag and Probe triggers for Run II, CMSHLT-95
  • Requesting to integrate an updated tau+MET trigger, CMSHLT-99
  • Displaced muon triggers for Run 2, CMSHLT-103
  • Addition of hadronic HLT paths from the SUSY PAG, CMSHLT-105

While the previous #5198 by @jalimena (also included here because it has not been merged yet) integrates:

  • CMSHLT-100 (Fix of relative isolation for taus)
  • CMSHLT-91 (rename WP80 as WP85 in top paths
  • CMSHLT-101, Addition of Single Jet Trigger with Trimming for B2G
  • CMSHLT-96 (control triggers for multijets)
  • CMSHLT-98 (ele/mu + jets for B2G)

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 6, 2014

A new Pull Request was created by @perrotta for CMSSW_7_2_X.

HLT integrations on top of 5198

It involves the following packages:

HLTrigger/Configuration

@Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard 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.

@perrotta
Copy link
Contributor Author

perrotta commented Sep 6, 2014

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 6, 2014

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests.

@perrotta
Copy link
Contributor Author

perrotta commented Sep 7, 2014

Pull Request updated with the following two tickets

  • CMSHLT-97 (displaced susy for exo)
  • CMSHLT-104 (met+btag for ZnnHbb)

and the corresponding FastSim customizations.

@perrotta
Copy link
Contributor Author

perrotta commented Sep 8, 2014

Uhm... Probably the message "This pull request can be automatically merged by project collaborators" just means that it merges now :-)
Perhaps I missed a different status of it before rebasing: I will check more carefully next time...

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 8, 2014

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 8, 2014

@perrotta
Copy link
Contributor Author

perrotta commented Sep 8, 2014

@nclopezo: since @lveldere already signed before rebasing, I think we can assume he would sign also after rebasing and merge this PR now, so that we can go ahed more easily with HLT integrations for pre6

nclopezo added a commit that referenced this pull request Sep 8, 2014
HLTrigger/Configuration -- HLT integrations on top of 5198
@nclopezo nclopezo merged commit 287d35a into cms-sw:CMSSW_7_2_X Sep 8, 2014
@lveldere
Copy link
Contributor

lveldere commented Sep 8, 2014

yes, feel free to proceed w/o my explicit re-approval in these kind of cases

On Mon, Sep 8, 2014 at 1:42 PM, perrotta notifications@github.com wrote:

@nclopezo https://github.com/nclopezo: since @lveldere
https://github.com/lveldere already signed before rebasing, I think we
can assume he would sign also after rebasing and merge this PR now, so that
we can go ahed more easily with HLT integrations for pre6


Reply to this email directly or view it on GitHub
#5202 (comment).

@perrotta
Copy link
Contributor Author

perrotta commented Sep 8, 2014

Thank you David and Lukas!

@perrotta perrotta deleted the hltIntegrationsOnTopOf5198 branch September 8, 2014 11:54
@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 8, 2014

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