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

Still some update to 1p4e34 menu 73X #7032

Closed

Conversation

perrotta
Copy link
Contributor

On top of #6877 (which is fully contained here, and can therefore get closed) this PR Integrates the following updates/fixes to the HLT menu:

  • CMSHLT-202: Modify HLT_PFJet40
  • CMSHLT-204: Various fixes for hadronic B2G paths (but revert the AK8Calo corrections because it looks like something is broken with it in the online GT's: to be updated once they will get fixed)
  • CMSHLT-215: Bug fix for B2G Mu + 2 Jets Path
  • Fixes HLT cfg dumps for TrackerGeometricDetESModule (same as Fix HLT cfg dumps for TrackerGeometricDetESModule (fallout of #6947) #7045 already merged in 74X)

@cmsbuild
Copy link
Contributor

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

Still some update to 1p4e34 menu 73X

It involves the following packages:

HLTrigger/Configuration

@Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @geoff-smith, @jalimena this is something you requested to watch as well.
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.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@perrotta
Copy link
Contributor Author

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 6, 2015

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 6, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 6, 2015

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 6, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 6, 2015

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

@Martin-Grunewald
Copy link
Contributor

@perrotta
let's please close this PR as it is now superseeded by #7116

@perrotta perrotta closed this Jan 12, 2015
@perrotta perrotta deleted the stillSomeUpdateTo1p4e34Menu_73X branch January 12, 2015 07:03
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

3 participants