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

Disable SETMuon and caloMuon from the standard reconstruction #14747

Merged
merged 8 commits into from Jul 5, 2016
Merged

Disable SETMuon and caloMuon from the standard reconstruction #14747

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8b48cb9
remove SETMuons from the standard reconstruction and validation
jhgoh Jun 2, 2016
330f415
disable caloMuons with very high pT threashold
jhgoh Jun 2, 2016
9ae012d
remove SETMuon from the path
jhgoh Jun 7, 2016
b7a3a7f
fix to apply caloMuon P cut on right place
jhgoh Jun 7, 2016
a64175c
fix fastsim
jhgoh Jun 8, 2016
6ea4e59
put back standAloneSETMuons for DT DQM to work, remove calomuons from…
jhgoh Jun 14, 2016
8c3b928
remove SETMuons from the DT DQM
jhgoh Jun 28, 2016
cc8e8e3
remove calomuon producer from the HI reco
jhgoh Jun 29, 2016
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
8 changes: 2 additions & 6 deletions RecoMuon/Configuration/python/RecoMuonPPonly_cff.py
View file
Open in desktop
Expand Up @@ -30,9 +30,6 @@
# TeV refinement
from RecoMuon.GlobalMuonProducer.tevMuons_cfi import *

# SET Muon tracking
from RecoMuon.Configuration.SETRecoMuon_cff import *

# Muon Id producer
from RecoMuon.MuonIdentification.muonIdProducerSequence_cff import *
muons1stStep.fillGlobalTrackQuality = True
Expand All @@ -58,9 +55,8 @@

# Muon Reconstruction plus Isolation
muonreco_plus_isolation = cms.Sequence(muonreco*muIsolation)
muonreco_plus_isolation_plus_SET = cms.Sequence(muonreco_plus_isolation*muonreco_with_SET)

muonrecoComplete = cms.Sequence(muonreco_plus_isolation_plus_SET*muonSelectionTypeSequence)
muonrecoComplete = cms.Sequence(muonreco_plus_isolation*muonSelectionTypeSequence)


# _-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_- #
Expand All @@ -72,7 +68,7 @@

#from RecoMuon.MuonIdentification.earlyMuons_cfi import earlyMuons

muonGlobalReco = cms.Sequence(globalmuontracking*muonIdProducerSequence*muonSelectionTypeSequence*muIsolation*muonreco_with_SET)
muonGlobalReco = cms.Sequence(globalmuontracking*muonIdProducerSequence*muonSelectionTypeSequence*muIsolation*muonreco)

# ... instead, the sequences will be run in the following order:
# 1st - standalonemuontracking
Expand Down
11 changes: 1 addition & 10 deletions RecoMuon/Configuration/python/RecoMuon_EventContent_cff.py
View file
Open in desktop
Expand Up @@ -72,16 +72,7 @@
'keep TrackingRecHitsOwned_cosmicMuons_*_*',
'keep recoTrackExtras_cosmicMuons1Leg_*_*',
'keep TrackingRecHitsOwned_cosmicMuons1Leg_*_*',
'keep recoTracks_cosmicsVetoTracks_*_*',
# SET muons
'keep *_SETMuonSeed_*_*',
'keep recoTracks_standAloneSETMuons_*_*',
'keep recoTrackExtras_standAloneSETMuons_*_*',
'keep TrackingRecHitsOwned_standAloneSETMuons_*_*',
'keep recoTracks_globalSETMuons_*_*',
'keep recoTrackExtras_globalSETMuons_*_*',
'keep TrackingRecHitsOwned_globalSETMuons_*_*',
'keep recoMuons_muonsWithSET_*_*')
'keep recoTracks_cosmicsVetoTracks_*_*')
)
# Full Event content
RecoMuonFEVT = cms.PSet(
Expand Down
2 changes: 1 addition & 1 deletion RecoMuon/MuonIdentification/python/calomuons_cfi.py
View file
Open in desktop
Expand Up @@ -12,7 +12,7 @@
inputMuons = cms.InputTag("muons1stStep"),
inputTracks = cms.InputTag("generalTracks"),
minCaloCompatibility = cms.double(0.6),
minPt = cms.double(1.0)
minPt = cms.double(1E9)
Copy link
Contributor

Choose a reason for hiding this comment

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

did you check if this does anything?
Looking at implementation of CaloMuonProducer (since 2009), most of the configuration parameters are actually bogus.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cut was applied on wrong place :-( that had to be applied on MuonIdProducer.

)


Expand Down
38 changes: 0 additions & 38 deletions Validation/RecoMuon/python/associators_cff.py
View file
Open in desktop
Expand Up @@ -87,24 +87,6 @@
label_tr = cms.InputTag('extractedGlobalMuons')
)

tpToStaSETTrackAssociation = cms.EDProducer('TrackAssociatorEDProducer',
associator = cms.InputTag('trackAssociatorByDeltaR'),
label_tp = cms.InputTag('mix', 'MergedTrackTruth'),
label_tr = cms.InputTag('standAloneSETMuons','')
)

tpToStaSETUpdTrackAssociation = cms.EDProducer('TrackAssociatorEDProducer',
associator = cms.InputTag('trackAssociatorByDeltaR'),
label_tp = cms.InputTag('mix', 'MergedTrackTruth'),
label_tr = cms.InputTag('standAloneSETMuons','UpdatedAtVtx')
)

tpToGlbSETTrackAssociation = cms.EDProducer('TrackAssociatorEDProducer',
associator = cms.InputTag('trackAssociatorByDeltaR'),
label_tp = cms.InputTag('mix', 'MergedTrackTruth'),
label_tr = cms.InputTag('globalSETMuons')
)

tpToTevFirstTrackAssociation = cms.EDProducer('TrackAssociatorEDProducer',
associator = cms.InputTag('trackAssociatorByDeltaR'),
label_tp = cms.InputTag('mix', 'MergedTrackTruth'),
Expand Down Expand Up @@ -173,9 +155,6 @@
tpToDisplacedStaSeedAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToDisplacedStaMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToDisplacedGlbMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToStaSETMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToStaSETUpdMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToGlbSETMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToTevFirstMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToTevPickyMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
tpToTevDytMuonAssociation = SimMuon.MCTruth.MuonAssociatorByHits_cfi.muonAssociatorByHits.clone()
Expand Down Expand Up @@ -241,21 +220,6 @@
tpToDisplacedGlbMuonAssociation.UseTracker = True
tpToDisplacedGlbMuonAssociation.UseMuon = True

tpToStaSETMuonAssociation.tpTag = 'mix:MergedTrackTruth'
tpToStaSETMuonAssociation.tracksTag = 'standAloneSETMuons'
tpToStaSETMuonAssociation.UseTracker = False
tpToStaSETMuonAssociation.UseMuon = True

tpToStaSETUpdMuonAssociation.tpTag = 'mix:MergedTrackTruth'
tpToStaSETUpdMuonAssociation.tracksTag = 'standAloneSETMuons:UpdatedAtVtx'
tpToStaSETUpdMuonAssociation.UseTracker = False
tpToStaSETUpdMuonAssociation.UseMuon = True

tpToGlbSETMuonAssociation.tpTag = 'mix:MergedTrackTruth'
tpToGlbSETMuonAssociation.tracksTag = 'globalSETMuons'
tpToGlbSETMuonAssociation.UseTracker = True
tpToGlbSETMuonAssociation.UseMuon = True

tpToTevFirstMuonAssociation.tpTag = 'mix:MergedTrackTruth'
tpToTevFirstMuonAssociation.tracksTag = 'tevMuons:firstHit'
tpToTevFirstMuonAssociation.UseTracker = True
Expand Down Expand Up @@ -410,8 +374,6 @@

muonAssociationRefit_seq = cms.Sequence(tpToStaRefitMuonAssociation+tpToStaRefitUpdMuonAssociation)

muonAssociationSET_seq = cms.Sequence(tpToStaSETMuonAssociation+tpToStaSETUpdMuonAssociation+tpToGlbSETMuonAssociation)

muonAssociationCosmic_seq = cms.Sequence(
tpToTkCosmicSelMuonAssociation+ tpToTkCosmic1LegSelMuonAssociation
+tpToStaCosmicSelMuonAssociation+tpToStaCosmic1LegSelMuonAssociation
Expand Down
52 changes: 2 additions & 50 deletions Validation/RecoMuon/python/muonValidation_cff.py
View file
Open in desktop
Expand Up @@ -43,27 +43,6 @@
glbMuonTrackVTrackAssoc.usetracker = True
glbMuonTrackVTrackAssoc.usemuon = True

staSETMuonTrackVTrackAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
staSETMuonTrackVTrackAssoc.associatormap = 'tpToStaSETTrackAssociation'
staSETMuonTrackVTrackAssoc.associators = ('trackAssociatorByDeltaR',)
staSETMuonTrackVTrackAssoc.label = ('standAloneSETMuons',)
staSETMuonTrackVTrackAssoc.usetracker = False
staSETMuonTrackVTrackAssoc.usemuon = True

staSETUpdMuonTrackVTrackAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
staSETUpdMuonTrackVTrackAssoc.associatormap = 'tpToStaSETUpdTrackAssociation'
staSETUpdMuonTrackVTrackAssoc.associators = ('trackAssociatorByDeltaR',)
staSETUpdMuonTrackVTrackAssoc.label = ('standAloneSETMuons:UpdatedAtVtx',)
staSETUpdMuonTrackVTrackAssoc.usetracker = False
staSETUpdMuonTrackVTrackAssoc.usemuon = True

glbSETMuonTrackVTrackAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
glbSETMuonTrackVTrackAssoc.associatormap = 'tpToGlbSETTrackAssociation'
glbSETMuonTrackVTrackAssoc.associators = ('trackAssociatorByDeltaR',)
glbSETMuonTrackVTrackAssoc.label = ('globalSETMuons',)
glbSETMuonTrackVTrackAssoc.usetracker = True
glbSETMuonTrackVTrackAssoc.usemuon = True

tevMuonFirstTrackVTrackAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
tevMuonFirstTrackVTrackAssoc.associatormap = 'tpToTevFirstTrackAssociation'
tevMuonFirstTrackVTrackAssoc.associators = ('trackAssociatorByDeltaR',)
Expand Down Expand Up @@ -186,27 +165,6 @@
displacedGlbMuonTrackVMuonAssoc.tipTP = cms.double(85.)
displacedGlbMuonTrackVMuonAssoc.lipTP = cms.double(210.)

staSETMuonTrackVMuonAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
staSETMuonTrackVMuonAssoc.associatormap = 'tpToStaSETMuonAssociation'
staSETMuonTrackVMuonAssoc.associators = ('MuonAssociationByHits',)
staSETMuonTrackVMuonAssoc.label = ('standAloneSETMuons',)
staSETMuonTrackVMuonAssoc.usetracker = False
staSETMuonTrackVMuonAssoc.usemuon = True

staSETUpdMuonTrackVMuonAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
staSETUpdMuonTrackVMuonAssoc.associatormap = 'tpToStaSETUpdMuonAssociation'
staSETUpdMuonTrackVMuonAssoc.associators = ('MuonAssociationByHits',)
staSETUpdMuonTrackVMuonAssoc.label = ('standAloneSETMuons:UpdatedAtVtx',)
staSETUpdMuonTrackVMuonAssoc.usetracker = False
staSETUpdMuonTrackVMuonAssoc.usemuon = True

glbSETMuonTrackVMuonAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
glbSETMuonTrackVMuonAssoc.associatormap = 'tpToGlbSETMuonAssociation'
glbSETMuonTrackVMuonAssoc.associators = ('MuonAssociationByHits',)
glbSETMuonTrackVMuonAssoc.label = ('globalSETMuons',)
glbSETMuonTrackVMuonAssoc.usetracker = True
glbSETMuonTrackVMuonAssoc.usemuon = True

tevMuonFirstTrackVMuonAssoc = Validation.RecoMuon.MuonTrackValidator_cfi.muonTrackValidator.clone()
tevMuonFirstTrackVMuonAssoc.associatormap = 'tpToTevFirstMuonAssociation'
tevMuonFirstTrackVMuonAssoc.associators = ('MuonAssociationByHits',)
Expand Down Expand Up @@ -436,12 +394,6 @@
+tpToDisplacedGlbMuonAssociation + displacedGlbMuonTrackVMuonAssoc
)

muonValidationSET_seq = cms.Sequence(
tpToStaSETMuonAssociation + staSETMuonTrackVMuonAssoc
+tpToStaSETUpdMuonAssociation + staSETUpdMuonTrackVMuonAssoc
+tpToGlbSETMuonAssociation + glbSETMuonTrackVMuonAssoc
)

muonValidationCosmic_seq = cms.Sequence(
tpToTkCosmicSelMuonAssociation + trkCosmicMuonTrackVSelMuonAssoc
+tpToTkCosmic1LegSelMuonAssociation + trkCosmic1LegMuonTrackVSelMuonAssoc
Expand All @@ -453,11 +405,11 @@

# The full offline muon validation sequence
recoMuonValidation = cms.Sequence(
muonValidation_seq + muonValidationTEV_seq + muonValidationRefit_seq + muonValidationDisplaced_seq + muonValidationSET_seq
muonValidation_seq + muonValidationTEV_seq + muonValidationRefit_seq + muonValidationDisplaced_seq
)

from Configuration.StandardSequences.Eras import eras
# no displaces or SET muons in fastsim
# no displaces in fastsim
if eras.fastSim.isChosen():
recoMuonValidation = cms.Sequence(muonValidation_seq + muonValidationTEV_seq + muonValidationRefit_seq)

Expand Down