-
Notifications
You must be signed in to change notification settings - Fork 4.6k
TICL-barrel: run CLUE in the barrel calorimeters and first workflows #47859
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
Changes from all commits
a3a327e
1d12a41
e1525fa
87e1e8d
ae10daa
d918b9c
f4e6d86
7793420
939fcb4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
| from .CE_E_Front_300um_cfi import generator | ||
|
|
||
| _pgunPSet = generator.PGunParameters | ||
|
|
||
| _pgunPSet.ControlledByEta = cms.bool(True) | ||
| _pgunPSet.MinEta = cms.double(-1.479) | ||
| _pgunPSet.MaxEta = cms.double(1.479) | ||
| _pgunPSet.RMin = cms.double(128.5) | ||
| _pgunPSet.RMax = cms.double(128.4) | ||
| _pgunPSet.ZMin = cms.double(-230) | ||
| _pgunPSet.ZMax = cms.double(230) | ||
|
|
||
| generator.PGunParameters = _pgunPSet |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| ticl_barrel = cms.Modifier() |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltBarrelLayerClustersEBL1Seeded = cms.EDProducer('BarrelLayerClusterProducer', | ||
| recHits = cms.InputTag("hltParticleFlowRecHitECALL1Seeded"), | ||
| plugin = cms.PSet( | ||
| outlierDeltaFactor = cms.double(2.7 * 0.0175), | ||
| kappa = cms.double(3.5), | ||
| maxLayerIndex = cms.int32(0), | ||
| deltac = cms.double(1.8 * 0.0175), | ||
| fractionCutoff = cms.double(0.0), | ||
| doSharing = cms.bool(False), | ||
| type = cms.string('EBCLUE') | ||
| ) | ||
| ) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If this is the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I agree, should I change the ones for HGCAL as well? E.g., hltHgcalLayerClustersHSi. |
||
|
|
||
| hltBarrelLayerClustersEB = cms.EDProducer('BarrelLayerClusterProducer', | ||
| recHits = cms.InputTag("hltParticleFlowRecHitECALUnseeded"), | ||
| plugin = cms.PSet( | ||
| outlierDeltaFactor = cms.double(2.7 * 0.0175), | ||
| kappa = cms.double(1), | ||
| maxLayerIndex = cms.int32(0), | ||
| deltac = cms.double(1.8 * 0.0175), | ||
| fractionCutoff = cms.double(0.0), | ||
| doSharing = cms.bool(False), | ||
| type = cms.string('EBCLUE') | ||
| ) | ||
| ) | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltBarrelLayerClustersHB = cms.EDProducer('BarrelLayerClusterProducer', | ||
| recHits = cms.InputTag("hltParticleFlowRecHitHBHE"), | ||
| plugin = cms.PSet( | ||
| outlierDeltaFactor = cms.double(2.7 * 0.0175), | ||
| kappa = cms.double(3.5), | ||
| maxLayerIndex = cms.int32(4), | ||
| deltac = cms.double(1.8 * 0.0175), | ||
| fractionCutoff = cms.double(0.0), | ||
| doSharing = cms.bool(False), | ||
| type = cms.string('HBCLUE') | ||
| ) | ||
| ) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,7 +2,7 @@ | |
|
|
||
| hltEgammaHGCALIDVarsL1Seeded = cms.EDProducer("EgammaHLTHGCalIDVarProducer", | ||
| hgcalRecHits = cms.InputTag("hltParticleFlowRecHitHGCL1Seeded"), | ||
| layerClusters = cms.InputTag("hltHgcalMergeLayerClustersL1Seeded"), | ||
| layerClusters = cms.InputTag("hltMergeLayerClustersL1Seeded"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure I follow the logic. If the TICL in the barrel
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, that is expected so that, at a later stage (not with this PR), it will be possible to build tracksters treating the calorimeters as a whole. The idea is that we do not want to have two different collections so that the indices returned from Trackster::vertices() will not be ambiguous. My understanding is that doing so should be safe for the subsequent steps in HGCAL, since the clusters used CLUE3D are coming from the tiles, so tracksters are still built in the endcaps only, for now. |
||
| rCylinder = cms.double(2.8), | ||
| recoEcalCandidateProducer = cms.InputTag("hltEgammaCandidatesL1Seeded") | ||
| ) | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,7 +2,7 @@ | |
|
|
||
| hltEgammaHGCALIDVarsUnseeded = cms.EDProducer("EgammaHLTHGCalIDVarProducer", | ||
| hgcalRecHits = cms.InputTag("hltParticleFlowRecHitHGC"), | ||
| layerClusters = cms.InputTag("hltHgcalMergeLayerClusters"), | ||
| layerClusters = cms.InputTag("hltMergeLayerClusters"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure I follow the logic. If the TICL in the barrel
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, that is expected so that, at a later stage (not with this PR), it will be possible to build tracksters treating the calorimeters as a whole. The idea is that we do not want to have two different collections so that the indices returned from My understanding is that doing so should be safe for the subsequent steps in HGCAL, since the clusters used CLUE3D are coming from the tiles, so tracksters are still built in the endcaps only, for now. |
||
| rCylinder = cms.double(2.8), | ||
| recoEcalCandidateProducer = cms.InputTag("hltEgammaCandidatesUnseeded") | ||
| ) | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,7 +5,7 @@ | |
| drMax = cms.double(0.2), | ||
| drVetoEM = cms.double(0.02), | ||
| drVetoHad = cms.double(0.0), | ||
| layerClusterProducer = cms.InputTag("hltHgcalMergeLayerClustersL1Seeded"), | ||
| layerClusterProducer = cms.InputTag("hltMergeLayerClustersL1Seeded"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment here. |
||
| minEnergyEM = cms.double(0.02), | ||
| minEnergyHad = cms.double(0.07), | ||
| recoEcalCandidateProducer = cms.InputTag("hltEgammaCandidatesL1Seeded"), | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,7 +5,7 @@ | |
| drMax = cms.double(0.2), | ||
| drVetoEM = cms.double(0.02), | ||
| drVetoHad = cms.double(0.0), | ||
| layerClusterProducer = cms.InputTag("hltHgcalMergeLayerClusters"), | ||
| layerClusterProducer = cms.InputTag("hltMergeLayerClusters"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment here. |
||
| minEnergyEM = cms.double(0.02), | ||
| minEnergyHad = cms.double(0.07), | ||
| recoEcalCandidateProducer = cms.InputTag("hltEgammaCandidatesUnseeded"), | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltFilteredLayerClustersCLUE3DHighL1Seeded = cms.EDProducer("FilteredLayerClustersProducer", | ||
| LayerClusters = cms.InputTag("hltHgcalMergeLayerClustersL1Seeded"), | ||
| LayerClustersInputMask = cms.InputTag("hltHgcalMergeLayerClustersL1Seeded","InitialLayerClustersMask"), | ||
| LayerClusters = cms.InputTag("hltMergeLayerClustersL1Seeded"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment here. Does that mean that clusters in input to |
||
| LayerClustersInputMask = cms.InputTag("hltMergeLayerClustersL1Seeded","InitialLayerClustersMask"), | ||
| clusterFilter = cms.string('ClusterFilterByAlgoAndSize'), | ||
| iteration_label = cms.string('CLUE3DHigh'), | ||
| max_cluster_size = cms.int32(9999), | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltFilteredLayerClustersCLUE3DHigh = cms.EDProducer("FilteredLayerClustersProducer", | ||
| LayerClusters = cms.InputTag("hltHgcalMergeLayerClusters"), | ||
| LayerClustersInputMask = cms.InputTag("hltHgcalMergeLayerClusters","InitialLayerClustersMask"), | ||
| LayerClusters = cms.InputTag("hltMergeLayerClusters"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment here. |
||
| LayerClustersInputMask = cms.InputTag("hltMergeLayerClusters","InitialLayerClustersMask"), | ||
| clusterFilter = cms.string('ClusterFilterByAlgoAndSize'), | ||
| iteration_label = cms.string('CLUE3DHigh'), | ||
| max_cluster_size = cms.int32(9999), | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltFilteredLayerClustersRecoveryL1Seeded = cms.EDProducer("FilteredLayerClustersProducer", | ||
| LayerClusters = cms.InputTag("hltHgcalMergeLayerClustersL1Seeded"), | ||
| LayerClusters = cms.InputTag("hltMergeLayerClustersL1Seeded"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here as well. |
||
| LayerClustersInputMask = cms.InputTag("hltTiclTrackstersCLUE3DHighL1Seeded"), | ||
| algo_number = cms.vint32(6, 7, 8), | ||
| clusterFilter = cms.string('ClusterFilterBySize'), | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| import FWCore.ParameterSet.Config as cms | ||
|
|
||
| hltFilteredLayerClustersRecovery = cms.EDProducer("FilteredLayerClustersProducer", | ||
| LayerClusters = cms.InputTag("hltHgcalMergeLayerClusters"), | ||
| LayerClusters = cms.InputTag("hltMergeLayerClusters"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. And here, too. |
||
| LayerClustersInputMask = cms.InputTag("hltTiclTrackstersCLUE3DHigh"), | ||
| algo_number = cms.vint32(6, 7, 8), | ||
| clusterFilter = cms.string('ClusterFilterBySize'), | ||
|
|
||
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please document also the
0.758offset?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right, I forgot about it, thanks for spotting it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another small request, can you add
29634.758to theph2_hltshort matrix, see #48262 for an example.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brusale can you address this comment too?