Enable DNN supercluster in Phase2 HLT #46010

RSalvatico · 2024-09-17T06:19:32Z

PR description:

Enable the usage of the DNN superclustering developed for the endcaps (PR #45333) in the Phase2 HLT reconstruction. In particular, this PR allows us to use:

Superclustering DNN (TracksterLinkingBySuperClusteringDNN)
Superclustering with Mustache inside TICL (TracksterLinkingbySuperClusteringMustache)

using process modifiers.

PR validation:

Successfully run several upgrade workflows such as
29691.203 SingleElectronPt15Eta1p7_2p7_2026D110 + ticl_v5
29691.204 SingleElectronPt15Eta1p7_2p7_2026D110 + ticl_v5_mustache

cmsbuild · 2024-09-17T06:20:00Z

cms-bot internal usage

cmsbuild · 2024-09-17T06:21:19Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46010/41813

There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Add run3 tracking low pu era #33532, CMSSW Integration of LST #45117, Introduce support for cluster splitting and JetCore iteration in Phase-2 track reconstruction #46001, FastSim premixing workflows and matrix cleanup #46007

cmsbuild · 2024-09-17T06:21:42Z

A new Pull Request was created by @RSalvatico for master.

It involves the following packages:

Configuration/PyReleaseValidation (upgrade, pdmv)
HLTrigger/Configuration (hlt)

@AdrianoDee, @Martin-Grunewald, @cmsbuild, @kskovpen, @miquork, @mmusich, @srimanob, @subirsarkar, @sunilUIET can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @SohamBhattacharya, @VourMa, @fabiocos, @makortel, @missirol, @mmusich, @rovere, @silviodonato, @slomeo this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

felicepantaleo · 2024-09-17T06:23:01Z

test parameters:

workflow_opts= -w upgrade
workflows = 29888.203,29688.203

felicepantaleo · 2024-09-17T06:23:39Z

@cmsbuild please test

felicepantaleo · 2024-09-17T06:29:48Z

@DebabrataBhowmik @waredjeb @rovere @VourMa fyi

cmsbuild · 2024-09-17T11:34:59Z

+1

Size: This PR adds an extra 68KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-eb053e/41547/summary.html
COMMIT: a85bda5
CMSSW: CMSSW_14_2_X_2024-09-16-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/46010/41547/install.sh to create a dev area with all the needed externals and cmssw changes.

DAS Queries: The DAS query tests failed, see the summary page for details.

Comparison Summary

Summary:

You potentially removed 2 lines from the logs
Reco comparison results: 10 differences found in the comparisons
DQMHistoTests: Total files compared: 46
DQMHistoTests: Total histograms compared: 3532200
DQMHistoTests: Total failures: 511
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 3531669
DQMHistoTests: Total skipped: 20
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 45 files compared)
Checked 202 log files, 171 edm output root files, 46 DQM output files
TriggerResults: found differences in 2 / 44 workflows

mmusich · 2024-09-17T11:37:45Z

enable hlt_p2_timing

mmusich

cosmetics

mmusich · 2024-09-17T07:15:13Z

HLTrigger/Configuration/python/HLT_75e33/modules/hltEgammaCandidatesL1Seeded_cfi.py

@@ -5,3 +5,6 @@
    scHybridBarrelProducer = cms.InputTag("hltParticleFlowSuperClusterECALL1Seeded","particleFlowSuperClusterECALBarrel"),
    scIslandEndcapProducer = cms.InputTag("hltParticleFlowSuperClusterHGCalFromTICLL1Seeded")
 )
+
+from Configuration.ProcessModifiers.ticl_v5_cff import ticl_v5
+ticl_v5.toModify(hltEgammaCandidatesL1Seeded, scIslandEndcapProducer = cms.InputTag("hltTiclEGammaSuperClusterProducerL1Seeded"))


missing newline.

mmusich · 2024-09-17T07:15:25Z

HLTrigger/Configuration/python/HLT_75e33/modules/hltEgammaCandidatesUnseeded_cfi.py

@@ -5,3 +5,6 @@
    scHybridBarrelProducer = cms.InputTag("hltParticleFlowSuperClusterECALUnseeded","particleFlowSuperClusterECALBarrel"),
    scIslandEndcapProducer = cms.InputTag("hltParticleFlowSuperClusterHGCalFromTICLUnseeded")
 )
+
+from Configuration.ProcessModifiers.ticl_v5_cff import ticl_v5
+ticl_v5.toModify(hltEgammaCandidatesUnseeded, scIslandEndcapProducer = cms.InputTag("hltTiclEGammaSuperClusterProducerUnseeded"))


missing newline.

mmusich · 2024-09-17T07:15:29Z

HLTrigger/Configuration/python/HLT_75e33/modules/hltTiclTracksterLinksL1Seeded_cfi.py

+    regressionAndPid = cms.bool(True),
+    tfDnnLabel = cms.string('tracksterSelectionTf'),
+    tracksters_collections = cms.VInputTag("hltTiclTrackstersCLUE3DHighL1Seeded", "hltTiclTrackstersPassthrough")
+)


missing newline.

mmusich · 2024-09-17T07:15:35Z

HLTrigger/Configuration/python/HLT_75e33/modules/hltTiclTracksterLinksUnseeded_cfi.py

+    regressionAndPid = cms.bool(True),
+    tfDnnLabel = cms.string('tracksterSelectionTf'),
+    tracksters_collections = cms.VInputTag("hltTiclTrackstersCLUE3DHigh", "hltTiclTrackstersPassthrough")
+)


missing newline.

mmusich · 2024-09-17T07:15:39Z

...guration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaL1SeededSequence_cfi.py

+                                  ticlSuperClusters=cms.InputTag("hltTiclTracksterLinksSuperclusteringMustacheL1Seeded"),
+                                  ticlTrackstersEM=cms.InputTag("hltTiclTrackstersCLUE3DHighL1Seeded"),
+                                  layerClusters=cms.InputTag("hltHgcalMergeLayerClustersL1Seeded")
+                                  )


missing newline.

mmusich · 2024-09-17T07:15:53Z

...guration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

+(alpaka & ticl_superclustering_mustache_ticl).toModify(hltTiclEGammaSuperClusterProducerUnseeded, 
+                                                       ticlSuperClusters=cms.InputTag("hltTiclTracksterLinksSuperclusteringMustacheUnseeded"),
+                                                       ticlTrackstersEM=cms.InputTag("hltTiclTrackstersCLUE3DHigh"),
+                                                       layerClusters=cms.InputTag("hltHgcalMergeLayerClusters"))


missing newline.

VourMa · 2024-09-17T12:28:44Z

...guration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

+(alpaka & ticl_superclustering_dnn).toReplaceWith(_HgcalLocalRecoUnseededSequence, 
+                    cms.Sequence(
+                        hltHgcalDigis
+                        + hltHGCalUncalibRecHit
+                        + hltHGCalRecHit+hltParticleFlowRecHitHGC
+                        + hltHgcalSoARecHitsProducer
+                        + hltHgcalSoARecHitsLayerClustersProducer
+                        + hltHgcalSoALayerClustersProducer
+                        + hltHgCalLayerClustersFromSoAProducer
+                        + hltHgcalLayerClustersHSci
+                        + hltHgcalLayerClustersHSi
+                        + hltHgcalMergeLayerClusters
+                        ) 
+)


How does this differ from

cmssw/HLTrigger/Configuration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

Lines 30 to 45 in a85bda5

# Alpaka

from Configuration.ProcessModifiers.alpaka_cff import alpaka

alpaka.toReplaceWith(_HgcalLocalRecoUnseededSequence,

cms.Sequence(

hltHgcalDigis

+ hltHGCalUncalibRecHit

+ hltHGCalRecHit+hltParticleFlowRecHitHGC

+ hltHgcalSoARecHitsProducer

+ hltHgcalSoARecHitsLayerClustersProducer

+ hltHgcalSoALayerClustersProducer

+ hltHgCalLayerClustersFromSoAProducer

+ hltHgcalLayerClustersHSci

+ hltHgcalLayerClustersHSi

+ hltHgcalMergeLayerClusters

)

)

?

Don't the aforementioned lines cover the heterogeneous workflow more in general? Maybe I am missing how the ticl_superclustering_dnn modifier changes anything here.

No, I think you're right. In fact all the (alpaka & ticl_superclustering) modifiers are pointless, as alpaka and ticl_superclustering modify independent sequences. Thanks

VourMa · 2024-09-17T12:29:23Z

...guration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

+(alpaka & ticl_superclustering_mustache_ticl).toReplaceWith(_HgcalLocalRecoUnseededSequence, 
+                    cms.Sequence(
+                        hltHgcalDigis
+                        + hltHGCalUncalibRecHit
+                        + hltHGCalRecHit+hltParticleFlowRecHitHGC
+                        + hltHgcalSoARecHitsProducer
+                        + hltHgcalSoARecHitsLayerClustersProducer
+                        + hltHgcalSoALayerClustersProducer
+                        + hltHgCalLayerClustersFromSoAProducer
+                        + hltHgcalLayerClustersHSci
+                        + hltHgcalLayerClustersHSi
+                        + hltHgcalMergeLayerClusters
+                        ) 
+)


Same comment as above.

VourMa · 2024-09-17T12:40:23Z

...guration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

+(alpaka & ticl_superclustering_dnn).toModify(hltHgcalMergeLayerClusters,
+                                             layerClustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer"),
+                                             time_layerclustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer", "timeLayerCluster"))


Similar comment as above when compared to lines

cmssw/HLTrigger/Configuration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

Lines 46 to 48 in a85bda5

alpaka.toModify(hltHgcalMergeLayerClusters,

layerClustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer"),

time_layerclustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer", "timeLayerCluster"))

Similarly applied to

cmssw/HLTrigger/Configuration/python/HLT_75e33/sequences/HLTHgcalTiclPFClusteringForEgammaUnseededSequence_cfi.py

Lines 142 to 144 in a85bda5

(alpaka & ticl_superclustering_mustache_ticl).toModify(hltHgcalMergeLayerClusters,

layerClustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer"),

time_layerclustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer", "timeLayerCluster"))

In general the "# Alpaka & DNN" part seems to be duplicating modifications from above, potentially making the maintenance of the code harder. Maybe try to simplify - unless there is some reason for the duplication which I failed to see (potentially having to do with the order the modifiers are applied)?

Yes indeed, I'll clean up.

RSalvatico · 2024-09-27T16:01:34Z

It worked @mmusich, thank you!

cmsbuild · 2024-09-27T16:05:34Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46010/41931

There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Add run3 tracking low pu era #33532, CMSSW Integration of LST #45117, Introduce support for cluster splitting and JetCore iteration in Phase-2 track reconstruction #46001, FastSim premixing workflows and matrix cleanup #46007, add variants to the phase-2 relval workflow running the HLT timing menu, up to step2 #46064, Add 2024 FastSim workflow #46068

cmsbuild · 2024-09-27T16:05:58Z

Pull request #46010 was updated. @AdrianoDee, @Martin-Grunewald, @Moanwar, @cmsbuild, @kskovpen, @miquork, @mmusich, @srimanob, @subirsarkar, @sunilUIET can you please check and sign again.

mmusich · 2024-09-27T16:06:45Z

@cmsbuild, please test

mmusich · 2024-09-27T17:18:51Z

HLTrigger/Configuration/python/HLT_75e33/modules/hltTiclTracksterLinksL1Seeded_cfi.py

+    detector = cms.string('HGCAL'),
+    eid_input_name = cms.string('input'),


something doesn't look right here, when comparing with the fillDescriptions of this producer:

cmssw/RecoHGCal/TICL/plugins/TracksterLinksProducer.cc

Lines 278 to 303 in eaefa4c

void TracksterLinksProducer::fillDescriptions(edm::ConfigurationDescriptions &descriptions) {

edm::ParameterSetDescription desc;

edm::ParameterSetDescription linkingDesc;

linkingDesc.addNode(edm::PluginDescription<TracksterLinkingPluginFactory>("type", "Skeletons", true));

// Inference Plugins

edm::ParameterSetDescription inferenceDesc;

inferenceDesc.addNode(edm::PluginDescription<TracksterInferenceAlgoFactory>("type", "TracksterInferenceByDNN", true));

desc.add<edm::ParameterSetDescription>("pluginInferenceAlgoTracksterInferenceByDNN", inferenceDesc);

edm::ParameterSetDescription inferenceDescCNNv4;

inferenceDescCNNv4.addNode(

edm::PluginDescription<TracksterInferenceAlgoFactory>("type", "TracksterInferenceByCNNv4", true));

desc.add<edm::ParameterSetDescription>("pluginInferenceAlgoTracksterInferenceByCNNv4", inferenceDescCNNv4);

desc.add<edm::ParameterSetDescription>("linkingPSet", linkingDesc);

desc.add<std::vector<edm::InputTag>>("tracksters_collections", {edm::InputTag("ticlTrackstersCLUE3DHigh")});

desc.add<std::vector<edm::InputTag>>("original_masks",

{edm::InputTag("hgcalMergeLayerClusters", "InitialLayerClustersMask")});

desc.add<edm::InputTag>("layer_clusters", edm::InputTag("hgcalMergeLayerClusters"));

desc.add<edm::InputTag>("layer_clustersTime", edm::InputTag("hgcalMergeLayerClusters", "timeLayerCluster"));

desc.add<bool>("regressionAndPid", false);

desc.add<std::string>("detector", "HGCAL");

desc.add<std::string>("propagator", "PropagatorWithMaterial");

desc.add<std::string>("inferenceAlgo", "TracksterInferenceByDNN");

descriptions.add("tracksterLinksProducer", desc);

}

I am getting runtime exceptions as:

----- Begin Fatal Exception 27-Sep-2024 18:45:08 CEST----------------------- An exception of category 'Configuration' occurred while [0] Constructing the EventProcessor [1] Validating configuration of module: class=TracksterLinksProducer label='hltTiclTracksterLinksSuperclusteringDNNL1Seeded' Exception Message: Illegal parameters found in configuration. The parameters are named: 'eid_input_name' 'eid_min_cluster_energy' 'eid_n_clusters' 'eid_n_layers' 'eid_output_name_energy' 'eid_output_name_id' 'tfDnnLabel' You could be trying to use parameter names that are not allowed for this plugin or they could be misspelled. ----- End Fatal Exception -------------------------------------------------

Yes, the problem is that with the PR #45821 the TracksterLinksProducer has changed and so have the related configs, which I had looked at to create this one (and the unseeded version). Will change them accordingly.

cmsbuild · 2024-09-27T18:17:18Z

-1

Failed Tests: RelVals HLTP2Timing
Size: This PR adds an extra 44KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-eb053e/41785/summary.html
COMMIT: 8b04f56
CMSSW: CMSSW_14_2_X_2024-09-27-1100/el8_amd64_gcc12
Additional Tests: HLT_P2_TIMING
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/46010/41785/install.sh to create a dev area with all the needed externals and cmssw changes.

DAS Queries: The DAS query tests failed, see the summary page for details.

RelVals

29691.204

----- Begin Fatal Exception 27-Sep-2024 20:09:04 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=TracksterLinksProducer label='hltTiclTracksterLinksSuperclusteringMustacheUnseeded'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'eid_input_name'
 'eid_min_cluster_energy'
 'eid_n_clusters'
 'eid_n_layers'
 'eid_output_name_energy'
 'eid_output_name_id'
 'tfDnnLabel'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

29691.203

----- Begin Fatal Exception 27-Sep-2024 20:09:07 CEST-----------------------
An exception of category 'Configuration' occurred while
   [0] Constructing the EventProcessor
   [1] Validating configuration of module: class=TracksterLinksProducer label='hltTiclTracksterLinksSuperclusteringDNNUnseeded'
Exception Message:
Illegal parameters found in configuration.  The parameters are named:
 'eid_input_name'
 'eid_min_cluster_energy'
 'eid_n_clusters'
 'eid_n_layers'
 'eid_output_name_energy'
 'eid_output_name_id'
 'tfDnnLabel'
You could be trying to use parameter names that are not
allowed for this plugin or they could be misspelled.
----- End Fatal Exception -------------------------------------------------

cmsbuild · 2024-09-27T20:02:52Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46010/41935

There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Add run3 tracking low pu era #33532, CMSSW Integration of LST #45117, Introduce support for cluster splitting and JetCore iteration in Phase-2 track reconstruction #46001, FastSim premixing workflows and matrix cleanup #46007, add variants to the phase-2 relval workflow running the HLT timing menu, up to step2 #46064, Add 2024 FastSim workflow #46068

cmsbuild · 2024-09-27T20:03:14Z

Pull request #46010 was updated. @AdrianoDee, @Martin-Grunewald, @Moanwar, @cmsbuild, @kskovpen, @miquork, @mmusich, @srimanob, @subirsarkar, @sunilUIET can you please check and sign again.

RSalvatico · 2024-09-27T20:08:29Z

@cmsbuild, please test

cmsbuild · 2024-09-27T23:40:27Z

-1

Failed Tests: HLTP2Timing
Size: This PR adds an extra 44KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-eb053e/41793/summary.html
COMMIT: 7d57246
CMSSW: CMSSW_14_2_X_2024-09-27-1100/el8_amd64_gcc12
Additional Tests: HLT_P2_TIMING
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/46010/41793/install.sh to create a dev area with all the needed externals and cmssw changes.

DAS Queries: The DAS query tests failed, see the summary page for details.

Comparison Summary

Summary:

You potentially removed 1 lines from the logs
ROOTFileChecks: Some differences in event products or their sizes found
Reco comparison results: 97 differences found in the comparisons
DQMHistoTests: Total files compared: 46
DQMHistoTests: Total histograms compared: 3532408
DQMHistoTests: Total failures: 3406
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 3528982
DQMHistoTests: Total skipped: 20
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 45 files compared)
Checked 203 log files, 171 edm output root files, 46 DQM output files
TriggerResults: found differences in 2 / 44 workflows

cmsbuild added this to the CMSSW_14_2_X milestone Sep 17, 2024

cmsbuild added hlt-pending pending-signatures tests-pending orp-pending pdmv-pending upgrade-pending code-checks-pending labels Sep 17, 2024

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 17, 2024

cmsbuild added tests-started and removed tests-pending labels Sep 17, 2024

cmsbuild added tests-approved and removed tests-started labels Sep 17, 2024

mmusich reviewed Sep 17, 2024

View reviewed changes

VourMa reviewed Sep 17, 2024

View reviewed changes

cmsbuild added tests-pending code-checks-pending and removed tests-approved code-checks-approved labels Sep 17, 2024

cmsbuild added code-checks-pending and removed code-checks-rejected labels Sep 27, 2024

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 27, 2024

cmsbuild added tests-started and removed tests-pending labels Sep 27, 2024

mmusich reviewed Sep 27, 2024

View reviewed changes

cmsbuild added tests-rejected and removed tests-started labels Sep 27, 2024

cmsbuild mentioned this pull request Sep 27, 2024

FastSim premixing workflows and matrix cleanup #46007

Open

update configs

7d57246

cmsbuild added tests-pending code-checks-pending and removed tests-rejected code-checks-approved labels Sep 27, 2024

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 27, 2024

cmsbuild added tests-started and removed tests-pending labels Sep 27, 2024

cmsbuild added tests-rejected and removed tests-started labels Sep 27, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Enable DNN supercluster in Phase2 HLT #46010

Enable DNN supercluster in Phase2 HLT #46010

RSalvatico commented Sep 17, 2024

cmsbuild commented Sep 17, 2024 •

edited

Loading

cmsbuild commented Sep 17, 2024

cmsbuild commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

cmsbuild commented Sep 17, 2024

mmusich commented Sep 17, 2024

mmusich left a comment

mmusich Sep 17, 2024

mmusich Sep 17, 2024

mmusich Sep 17, 2024

mmusich Sep 17, 2024

mmusich Sep 17, 2024

mmusich Sep 17, 2024

VourMa Sep 17, 2024

RSalvatico Sep 17, 2024 •

edited

Loading

VourMa Sep 17, 2024

VourMa Sep 17, 2024

RSalvatico Sep 17, 2024

RSalvatico commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

mmusich commented Sep 27, 2024

mmusich Sep 27, 2024

RSalvatico Sep 27, 2024

cmsbuild commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

RSalvatico commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

	# Alpaka
	from Configuration.ProcessModifiers.alpaka_cff import alpaka
	alpaka.toReplaceWith(_HgcalLocalRecoUnseededSequence,
	cms.Sequence(
	hltHgcalDigis
	+ hltHGCalUncalibRecHit
	+ hltHGCalRecHit+hltParticleFlowRecHitHGC
	+ hltHgcalSoARecHitsProducer
	+ hltHgcalSoARecHitsLayerClustersProducer
	+ hltHgcalSoALayerClustersProducer
	+ hltHgCalLayerClustersFromSoAProducer
	+ hltHgcalLayerClustersHSci
	+ hltHgcalLayerClustersHSi
	+ hltHgcalMergeLayerClusters
	)
	)

	alpaka.toModify(hltHgcalMergeLayerClusters,
	layerClustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer"),
	time_layerclustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer", "timeLayerCluster"))

	(alpaka & ticl_superclustering_mustache_ticl).toModify(hltHgcalMergeLayerClusters,
	layerClustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer"),
	time_layerclustersEE = cms.InputTag("hltHgCalLayerClustersFromSoAProducer", "timeLayerCluster"))

		detector = cms.string('HGCAL'),
		eid_input_name = cms.string('input'),

	void TracksterLinksProducer::fillDescriptions(edm::ConfigurationDescriptions &descriptions) {
	edm::ParameterSetDescription desc;
	edm::ParameterSetDescription linkingDesc;
	linkingDesc.addNode(edm::PluginDescription<TracksterLinkingPluginFactory>("type", "Skeletons", true));
	// Inference Plugins
	edm::ParameterSetDescription inferenceDesc;
	inferenceDesc.addNode(edm::PluginDescription<TracksterInferenceAlgoFactory>("type", "TracksterInferenceByDNN", true));
	desc.add<edm::ParameterSetDescription>("pluginInferenceAlgoTracksterInferenceByDNN", inferenceDesc);

	edm::ParameterSetDescription inferenceDescCNNv4;
	inferenceDescCNNv4.addNode(
	edm::PluginDescription<TracksterInferenceAlgoFactory>("type", "TracksterInferenceByCNNv4", true));
	desc.add<edm::ParameterSetDescription>("pluginInferenceAlgoTracksterInferenceByCNNv4", inferenceDescCNNv4);

	desc.add<edm::ParameterSetDescription>("linkingPSet", linkingDesc);
	desc.add<std::vector<edm::InputTag>>("tracksters_collections", {edm::InputTag("ticlTrackstersCLUE3DHigh")});
	desc.add<std::vector<edm::InputTag>>("original_masks",
	{edm::InputTag("hgcalMergeLayerClusters", "InitialLayerClustersMask")});
	desc.add<edm::InputTag>("layer_clusters", edm::InputTag("hgcalMergeLayerClusters"));
	desc.add<edm::InputTag>("layer_clustersTime", edm::InputTag("hgcalMergeLayerClusters", "timeLayerCluster"));
	desc.add<bool>("regressionAndPid", false);
	desc.add<std::string>("detector", "HGCAL");
	desc.add<std::string>("propagator", "PropagatorWithMaterial");
	desc.add<std::string>("inferenceAlgo", "TracksterInferenceByDNN");
	descriptions.add("tracksterLinksProducer", desc);
	}

Enable DNN supercluster in Phase2 HLT #46010

Are you sure you want to change the base?

Enable DNN supercluster in Phase2 HLT #46010

Conversation

RSalvatico commented Sep 17, 2024

PR description:

PR validation:

cmsbuild commented Sep 17, 2024 • edited Loading

cmsbuild commented Sep 17, 2024

cmsbuild commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

felicepantaleo commented Sep 17, 2024

cmsbuild commented Sep 17, 2024

Comparison Summary

mmusich commented Sep 17, 2024

mmusich left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

RSalvatico Sep 17, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

RSalvatico commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

mmusich commented Sep 27, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cmsbuild commented Sep 27, 2024

RelVals

cmsbuild commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

RSalvatico commented Sep 27, 2024

cmsbuild commented Sep 27, 2024

Comparison Summary

cmsbuild commented Sep 17, 2024 •

edited

Loading

RSalvatico Sep 17, 2024 •

edited

Loading