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

Update Higgs Dalitz paths (75X) #11342

Conversation

jasperlauwers
Copy link
Contributor

backport of #11340

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @jasperlauwers for CMSSW_7_5_X.

Update Higgs Dalitz paths (75X)

It involves the following packages:

HLTriggerOffline/Higgs

@cmsbuild, @danduggan, @broen, @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.

@perrotta
Copy link
Contributor

@jasperlauwers : only the last commit in this ticket is actually the backport of #11340.
Could you please make separate pull requests for separate updates/fixes?
As such you would speed up the merging of the part which relates to the additional monitoring of the new Dalitz path(s), and as a consequence the integration of the new paths in the HLT menu.

@deguio
Copy link
Contributor

deguio commented Sep 24, 2015

please test

@deguio
Copy link
Contributor

deguio commented Sep 24, 2015

+1

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.

@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 after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

1 similar comment
@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 after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@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 after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 5, 2015

Pull request #11342 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

@jasperlauwers
Copy link
Contributor Author

The last backport for now #11198

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 6, 2015

Pull request #11342 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

@jasperlauwers jasperlauwers force-pushed the HLTriggerOffline_Higgs_Dalitz_75 branch from 12679f1 to 5100bb6 Compare October 9, 2015 11:04
@jasperlauwers
Copy link
Contributor Author

The PR has been updated to contain the commits that have been validated in 7_6_0_pre6.

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 9, 2015

Pull request #11342 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

@deguio
Copy link
Contributor

deguio commented Oct 10, 2015

please test

@deguio
Copy link
Contributor

deguio commented Oct 10, 2015

+1

@cmsbuild
Copy link
Contributor

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

@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 after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

@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 (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

@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 (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@davidlange6
Copy link
Contributor

Closing - not needed for HeavyIons (only planned 75x usage)

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.

6 participants