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

Further integrations in the HLT Run2 menu #4878

Merged
merged 7 commits into from Aug 5, 2014
Merged

Conversation

perrotta
Copy link
Contributor

@perrotta perrotta commented Aug 5, 2014

It evolves from #4783, with the latest additions to the Run2 HLT menu:

  • allow the BPH paths of CMSHLT-82 in the GRun menu (since the validation module is arrived)
  • integrate "HZZ2l2nu Photon + Jet Control trigger menus" of CMSHLT-83 in the FULL menu
  • integrate "NoBPTX paths for exo" of CMSHLT-87 in the FULL menu
  • integrate "High Pt Double Electron and Photon Paths for EXO" of CMSHLT-88 in the FULL menu
  • integrate "MultiJet Path for Exo" of CMSHLT-89 in the FULL menu

(the paths from the latest tickets will be promoted to the GRun subtables only when the corresponding DQM/Validation modules will be also provided).

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2014

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

Further integrations in the HLT Run2 menu

It involves the following packages:

HLTrigger/Configuration

@perrotta, @cmsbuild, @nclopezo, @Martin-Grunewald, @fwyzard, @Degano 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 Aug 5, 2014

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 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.

@Martin-Grunewald
Copy link
Contributor

Hi,

How did you manage to run the tests as they all fail (for me at least)
in 72X?

On Tue, 5 Aug 2014, perrotta wrote:

+1


Reply to this email directly or view it on GitHub:
#4878 (comment)

Martin

Martin W. Gruenewald e-mail: Martin.Grunewald@cern.ch
http://cern.ch/Martin.Grunewald

perrotta added a commit to perrotta/cmssw that referenced this pull request Aug 5, 2014
@perrotta
Copy link
Contributor Author

perrotta commented Aug 5, 2014

Hi Martin.
They fail also to me (in CMSSW_7_2_X_2014-08-05-0200), but I tested
the very same configurations already in an older IB
(CMSSW_7_2_X_2014-08-04-0200). Therefore, I know that the
configurations are fine. As soon as this annoying issue with the input
files is over, tests will run succesfully also in the newer IB's
A.

Martin Grunewald notifications@github.com ha scritto:

Hi,

How did you manage to run the tests as they all fail (for me at least)
in 72X?

On Tue, 5 Aug 2014, perrotta wrote:

+1


Reply to this email directly or view it on GitHub:
#4878 (comment)

Martin

Martin W. Gruenewald e-mail: Martin.Grunewald@cern.ch
http://cern.ch/Martin.Grunewald


Reply to this email directly or view it on GitHub:
#4878 (comment)


This message was sent using IMP, the Internet Messaging Program.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2014

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2014

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine).

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2014

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 5, 2014

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine).

ktf added a commit that referenced this pull request Aug 5, 2014
Further integrations in the HLT Run2 menu
@ktf ktf merged commit ed76a76 into cms-sw:CMSSW_7_2_X Aug 5, 2014
nclopezo added a commit that referenced this pull request Aug 6, 2014
HLTrigger/Configuration -- Fastsim configs were not derived properly in #4878
@perrotta perrotta deleted the cmsHLT83 branch August 7, 2014 09:23
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