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

Jump to bottom

Ecal egamma hlt #866

Merged
merged 25 commits into from Sep 20, 2013
Merged

Ecal egamma hlt #866

merged 25 commits into from Sep 20, 2013

Conversation

Martin-Grunewald
Copy link
Contributor

Pull request with HLT changes, ConfDB parsing updates, fixes to Ecal and Egamma migrations
so that HLT works, and including pull requests 778, 825, 845, 850, 851.

argiro and others added 25 commits August 14, 2013 16:59
@argiro
consumes migration
70ab44c
@argiro
consumes migration
5bc75b2
@argiro
consumes migration
2b3c54d
@argiro
cleanup old modules
34dc993
@argiro
removed transform, will be commited by someone else
f728edb
@argiro
removed spurioius file
74a80a6
@argiro
make consumes migration cope with RawToRecHit bulge
1348a2a
@argiro
Merged argiro/consumesmigration from repository argiro
18dc826 
Conflicts:
	RecoEcal/EgammaClusterProducers/interface/EgammaSimpleAnalyzer.h
	RecoEcal/EgammaClusterProducers/interface/ExampleClusterProducer.h
	RecoEcal/EgammaClusterProducers/interface/PreshowerAnalyzer.h
	RecoEcal/EgammaClusterProducers/src/EgammaSimpleAnalyzer.cc
	RecoEcal/EgammaClusterProducers/src/PreshowerAnalyzer.cc
@argiro
chances for consumes migration
949795b
@argiro
reverted, let HLT people change it
b945a95
@matteosan1
migration to consumers
6eb561a
@Martin-Grunewald
Bugfix in HLTrigger/special filters
096007f
@Martin-Grunewald
restore py file needed for HLT ConfDB Parsing
ebf1c39
@Martin-Grunewald
Add cfi file for DQMStore for use with HLT ConfDB parsing
2df96e9
@Martin-Grunewald
Merged refs/pull/845/head from repository cms-sw
bb7af88
@Martin-Grunewald
Fix Fix Fix
441ace7
@Martin-Grunewald
Merged refs/pull/778/head from repository cms-sw
8426b27
@Martin-Grunewald
Merged refs/pull/825/head from repository cms-sw
fbc4f38
@Martin-Grunewald
Merged refs/pull/845/head from repository cms-sw
1fc2919
@Martin-Grunewald
Merged refs/pull/850/head from repository cms-sw
b03b9ac
@Martin-Grunewald
Merged refs/pull/851/head from repository cms-sw
e68bbcb
@Martin-Grunewald
HLT modifs for EcalEgamma strings-to-InputTag migration
de43c88
@argiro
Added WorkerSimpleOld and WorkerWeightsOld for bkw compat
6afdba0
@Martin-Grunewald
Merged refs/pull/778/head from repository cms-sw
bbf9934
@Martin-Grunewald
More fies to this saga
8aa534d
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @Martin-Grunewald (Martin Grunewald) for CMSSW_7_0_X.

Ecal egamma hlt

It involves the following packages:

RecoEgamma/EgammaHLTProducers
Validation/EcalClusters
HLTrigger/special
HLTrigger/Configuration
RecoLocalCalo/EcalRecProducers
RecoEcal/EgammaClusterProducers
EventFilter/EcalRawToDigi
DQMServices/Core
RecoBTag/SoftLepton

@perrotta, @nclopezo, @danduggan, @fwyzard, @rovere, @Martin-Grunewald, @thspeer, @deguio, @slava77 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.
@ktf you are the release manager for this.

@Martin-Grunewald
Copy link
Contributor Author

+1

Martin

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

@argiro argiro mentioned this pull request Sep 18, 2013
Closed
@nclopezo
Copy link
Contributor

+1
https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc481/575/consoleFull
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/575

@thspeer
Copy link
Contributor

thspeer commented Sep 19, 2013

+1
Tested 8aa534d
No difference in reco, based on Jenkins tests

@deguio
Copy link
Contributor

deguio commented Sep 19, 2013

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it?

ktf added a commit that referenced this pull request Sep 20, 2013
@ktf
Merge pull request #866 from Martin-Grunewald/EcalEgammaHLT
c201a4b 
MIgration to consumes -- Ecal egamma HLT consumes migration, plus other ConfDB changes.
@ktf ktf merged commit c201a4b into cms-sw:CMSSW_7_0_X Sep 20, 2013
@thspeer thspeer mentioned this pull request Oct 4, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dqm-approved fully-signed hlt-approved reconstruction-approved tests-approved
Projects
None yet
Milestone
CMSSW_7_0_0_pre5
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@Martin-Grunewald @cmsbuild @nclopezo @thspeer @deguio @ktf @argiro @matteosan1