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.

Sign up for GitHub

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

GenJetFlavourInfos: fix compiler warning, use prunedGenParticles in associations to correctly match pat jet flavour info #20281

Merged
merged 5 commits into from Sep 1, 2017
Merged

GenJetFlavourInfos: fix compiler warning, use prunedGenParticles in associations to correctly match pat jet flavour info #20281

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8e3f660
changed to loop over unsigned int
Andrej-CMS Aug 28, 2017
7d315d0
code checks: use override instead of virtual
Andrej-CMS Aug 28, 2017
b26fa60
uint -> unsigned int
Andrej-CMS Aug 29, 2017
c7111f2
slimmedGenJetsFlavourInfos requires existing RefVectors to Hadrons fo…
Andrej-CMS Aug 29, 2017
be715ba
fixed indentation
Andrej-CMS Aug 30, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 6 additions & 5 deletions PhysicsTools/PatAlgos/plugins/GenJetFlavourInfoPreserver.cc
View file
Open in desktop
Expand Up @@ -27,9 +27,9 @@ namespace pat {
class GenJetFlavourInfoPreserver : public edm::stream::EDProducer<> {
public:
explicit GenJetFlavourInfoPreserver(const edm::ParameterSet & iConfig);
virtual ~GenJetFlavourInfoPreserver() { }
~GenJetFlavourInfoPreserver() override { }

virtual void produce(edm::Event & iEvent, const edm::EventSetup & iSetup) override;
void produce(edm::Event & iEvent, const edm::EventSetup & iSetup) override;

private:
const edm::EDGetTokenT<edm::View<reco::GenJet> > genJetsToken_;
Expand Down Expand Up @@ -72,12 +72,13 @@ pat::GenJetFlavourInfoPreserver::produce(edm::Event & iEvent, const edm::EventSe

edm::Ref<std::vector<reco::GenJet> > slimmedGenJetRef;

int i = -1;
for (auto const& genJet : *genJets){
i++;

for (uint i=0; i<genJets->size();++i){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi @Andrej-CMS - please change uint to unsigned int


slimmedGenJetRef = (*slimmedGenJetAssociation)[genJets->refAt(i)];
if(!slimmedGenJetRef) continue;
(*jetFlavourInfos)[reco::JetBaseRef(slimmedGenJetRef)] = (*genJetFlavourInfos)[i].second;

}

iEvent.put(std::move(jetFlavourInfos));
Expand Down