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 to tau iso by pt in the hlt config #13190

Merged
merged 1 commit into from Feb 5, 2016

Conversation

perrotta
Copy link
Contributor

@perrotta perrotta commented Feb 4, 2016

HLT configuration changes that take into account the modifications of the L2TauPixelIsoTagProducer, already merged in 80X with #13176

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

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

It involves the following packages:

HLTrigger/Configuration
RecoEgamma/EgammaElectronProducers
RecoEgamma/EgammaPhotonProducers
RecoHI/HiTracking
RecoLocalTracker/SiStripClusterizer
RecoTauTag/HLTProducers
RecoTracker/ConversionSeedGenerators
RecoTracker/MeasurementDet
RecoTracker/TkTrackingRegions
TrackingTools/GsfTracking
TrackingTools/KalmanUpdators
TrackingTools/TrajectoryFiltering

@perrotta, @cmsbuild, @cvuosalo, @slava77, @Martin-Grunewald, @fwyzard, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @forthommel, @abbiendi, @mandrenguyen, @nickmccoll, @lgray, @threus, @geoff-smith, @battibass, @makortel, @jhgoh, @dgulhan, @jlagram, @OlivierBondu, @yduhm, @yetkinyilmaz, @Sam-Harper, @GiacomoSguazzoni, @rovere, @VinInn, @bellan, @jalimena, @cerati, @mschrode, @richard-cms, @echapon, @trocino, @gbenelli, @jazzitup, @yenjie, @Martin-Grunewald, @kurtejung, @gpetruc, @istaslis this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@perrotta
Copy link
Contributor Author

perrotta commented Feb 4, 2016

Uhm... Extra signatures sneaked in because I attached two already merged pull requests (for testing purposes). I will rebase to get rid of them: only HLT must remain

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

Pull request #13190 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @davidlange6, @fwyzard can you please check and sign again.

@perrotta
Copy link
Contributor Author

perrotta commented Feb 4, 2016

Rebased to CMSSW_8_0_X_2016-02-04-1100
Only HLT signature is needed now

@perrotta
Copy link
Contributor Author

perrotta commented Feb 4, 2016

+1
(Here it is)

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@perrotta
Copy link
Contributor Author

perrotta commented Feb 4, 2016

@davidlange6
please consider this for pre6

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

-1

Tested at: 8e0384c
I found errors in the following unit tests:

---> test testRecoMETMETProducers had ERRORS

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13190/10998/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
77cf558
ce79f96
ec1fee3
49024f5
2365b6e
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13190/10998/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13190/10998/git-merge-result

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 4, 2016

@Martin-Grunewald
Copy link
Contributor

error unrelated to this PR

@Martin-Grunewald
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 5, 2016

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 5, 2016

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 5, 2016

@davidlange6
Copy link
Contributor

+1

cmsbuild added a commit that referenced this pull request Feb 5, 2016
Update to tau iso by pt in the hlt config
@cmsbuild cmsbuild merged commit 6582836 into cms-sw:CMSSW_8_0_X Feb 5, 2016
@perrotta perrotta deleted the updateToTauIsoByPtInHlt branch February 6, 2016 08:49
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