RPCMuon bugfix in MuonIdProducer

The RPCMuon is mis-implemented as a subset of TrackerMuons.
Originally it is designed to be a complementary ones to the TM's.
Looks wrong. What Slava says sounds reasonable. The RPC muon type should be
set more carefully.

Dima

P.S. I'm traveling so don't expect my quick response


On Tue, Aug 12, 2014 at 4:11 PM, Slava Krutelyov <notifications@github.com>
wrote:

> In RecoMuon/MuonIdentification/plugins/MuonIdProducer.cc:
>
> > @@ -587,7 +587,7 @@ void MuonIdProducer::produce(edm::Event& iEvent, const edm::EventSetup& iSetup)
> >  	     {
> >  		// make muon
> >  	       reco::Muon trackerMuon( makeMuon(iEvent, iSetup, reco::TrackRef( innerTrackCollectionHandle_, i ), reco::Muon::InnerTrack ) );
> > -		trackerMuon.setType( reco::Muon::TrackerMuon | reco::Muon::RPCMuon );
> > +		trackerMuon.setType( reco::Muon::TrackerMuon );
>
> @drkovalskyi <https://github.com/drkovalskyi>
> please check
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-sw/cmssw/pull/4902/files#r16111391>.
>