Skip to content

Phase2 Single_Tau_Trigger Path Added#49637

Open
Chihwan-An wants to merge 3 commits into
cms-sw:masterfrom
Chihwan-An:SingleTau_Trigger_v1
Open

Phase2 Single_Tau_Trigger Path Added#49637
Chihwan-An wants to merge 3 commits into
cms-sw:masterfrom
Chihwan-An:SingleTau_Trigger_v1

Conversation

@Chihwan-An

Copy link
Copy Markdown

Added Single Tau Trigger path for Phase2 , worked with Tau HLT Phase2 development group.

It was tested in miniAOD in CMSSW_15_1_0_pre4 and NanoAOD in CMSSW_16_0_0_pre2 , with Z'to tautau samples.

Trigger Efficiency had 48.7% .

  • Phase2 Single_Tau_Trigger Path Added #49569 was submitted before.
    • MinEta for HLT1PFTau EDFilters has a default value of -1.0, which the config understands it as being -2.1.
  • Slides are not shared with TSG group yet , shared with Tau HLT phase2 group .

@cmsbuild

cmsbuild commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

cms-bot internal usage

@cmsbuild

Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49637/47177

  • Found files with invalid states:
    • HLTrigger/Configuration/python/HLT_75e33/modules/hltPreLooseDeepTauPFTauHPS150L2NNeta2p1_cfi.py:

@cmsbuild

Copy link
Copy Markdown
Contributor

A new Pull Request was created by @Chihwan-An for master.

It involves the following packages:

  • HLTrigger/Configuration (hlt)

@Martin-Grunewald, @cmsbuild, @mmusich can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @SohamBhattacharya, @VourMa, @missirol, @mmusich, @rovere this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@Martin-Grunewald

Copy link
Copy Markdown
Contributor

please test

@cmsbuild

Copy link
Copy Markdown
Contributor

+1

Size: This PR adds an extra 48KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3976a9/49993/summary.html
COMMIT: 57b17b8
CMSSW: CMSSW_16_0_X_2025-12-15-2300/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/49637/49993/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially added 1 lines to the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 15 differences found in the comparisons
  • Reco comparison had 4 failed jobs
  • DQMHistoTests: Total files compared: 53
  • DQMHistoTests: Total histograms compared: 4280229
  • DQMHistoTests: Total failures: 9
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 4280200
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 52 files compared)
  • Checked 227 log files, 198 edm output root files, 53 DQM output files
  • TriggerResults: no differences found

@mmusich

mmusich commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

test parameters:

  • enable = hlt_p2_integration, hlt_p2_timing
  • workflows = ph2_hlt

@mmusich

mmusich commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

@cmsbuild, please test

@mmusich

mmusich commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Hi @Chihwan-An,
the timing tests are not completed, but I see that this PR is already failing one of the relval tests: 34434.758_TTbar_14TeV+Run4D121_HLT75e33TimingTiclBarrel, see log.
Can you try to reproduce locally by executing runTheMatrix.py -l 34434.758 locally and fix the configuration? From the exception message it looks like it should be fixable by re-ordering the modules scheduled in the new path HLT_LooseDeepTauPFTauHPS150_L2NN_eta2p1

@mmusich

mmusich commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

hold

  • Slides are not shared with TSG group yet , shared with Tau HLT phase2 group .

  • the new path should be discussed first in a HLT upgrade meeting @rovere @VourMa FYI

@cmsbuild

Copy link
Copy Markdown
Contributor

Pull request has been put on hold by @mmusich
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

@cmsbuild cmsbuild added the hold label Dec 16, 2025
@mmusich

mmusich commented Dec 17, 2025

Copy link
Copy Markdown
Contributor

@smuzaffar the hlt phase2 timing tests doesn't seem to start, can you check what is the problem with it?

@smuzaffar

Copy link
Copy Markdown
Contributor

@mmusich , I see that job started an hour ago (https://cmssdt.cern.ch/jenkins/job/ib-run-pr-hlt_p2_timing/241/console) . It should finish soon

@cmsbuild

Copy link
Copy Markdown
Contributor

-1

Failed Tests: RelVals
Size: This PR adds an extra 48KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3976a9/49995/summary.html
COMMIT: 57b17b8
CMSSW: CMSSW_16_0_X_2025-12-15-2300/el8_amd64_gcc13
Additional Tests: HLT_P2_INTEGRATION,HLT_P2_TIMING
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/49637/49995/install.sh to create a dev area with all the needed externals and cmssw changes.

HLT P2 Timing: chart

Failed RelVals

----- Begin Fatal Exception 16-Dec-2025 11:15:10 CET-----------------------
An exception of category 'ScheduleExecutionFailure' occurred while
   [0] Calling beginJob
Exception Message:
Unrunnable schedule
Dependent module later on Path
  module 'hltParticleFlowRecHitECALUnseeded' depends on 'hltEcalRecHit' which is later on path HLT_LooseDeepTauPFTauHPS150_L2NN_eta2p1
----- End Fatal Exception -------------------------------------------------

@cmsbuild

Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49637/48496

@cmsbuild

Copy link
Copy Markdown
Contributor

Pull request #49637 was updated. @Martin-Grunewald, @cmsbuild, @mmusich can you please check and sign again.

@mmusich

mmusich commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

@cmsbuild, please test

inputTag = cms.InputTag("hltHpsSelectedPFTauLooseTauWPDeepTau"),
saveTags = cms.bool(True),
triggerType = cms.int32(84)
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

picky but newline is missing.

+ hltHpsSelectedPFTauLooseTauWPDeepTau
+ hltHpsPFTau150LooseTauWPDeepTau
+ HLTEndSequence
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

picky but newline is missing.

@cmsbuild

Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49637/48497

@cmsbuild

Copy link
Copy Markdown
Contributor

Pull request #49637 was updated. @Martin-Grunewald, @cmsbuild, @mmusich can you please check and sign again.

@mmusich

mmusich commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

please test

@cmsbuild

Copy link
Copy Markdown
Contributor

+1

Size: This PR adds an extra 52KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3976a9/51936/summary.html
COMMIT: 198c6ab
CMSSW: CMSSW_16_1_X_2026-03-12-2300/el8_amd64_gcc13
Additional Tests: HLT_P2_INTEGRATION,HLT_P2_TIMING
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/49637/51936/install.sh to create a dev area with all the needed externals and cmssw changes.

HLT P2 Timing: chart

Comparison Summary

Summary:

  • No significant changes to the logs found
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 25 differences found in the comparisons
  • DQMHistoTests: Total files compared: 66
  • DQMHistoTests: Total histograms compared: 4566828
  • DQMHistoTests: Total failures: 278
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 4566530
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 65 files compared)
  • Checked 274 log files, 234 edm output root files, 66 DQM output files
  • TriggerResults: found differences in 1 / 64 workflows

@mmusich

mmusich commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

Updated timing cost of the additions:

Phase2 timing menu (on GPU)

CMSSW_16_1_X_2026-03-12-2300 CMSSW_16_1_X_2026-03-12-2300 + this PR
Screenshot from 2026-03-13 23-30-13 Screenshot from 2026-03-13 23-30-02

concentrated in the Tau, HGcal and Tracking categories:

plot(8)

From 6245.8 ms/ev to 6378.1 ms/ev , thus a +2.11% increase.

@cmsbuild

cmsbuild commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Milestone for this pull request has been moved to CMSSW_17_0_X. Please open a backport if it should also go in to CMSSW_16_1_X.

@cmsbuild

cmsbuild commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Milestone for this pull request has been moved to CMSSW_20_0_X. Please open a backport if it should also go in to CMSSW_17_0_X.

@makortel

makortel commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

(I assume this PR does not need a backport to 17_0_X (Run 3 legacy))

@@ -2,9 +2,12 @@

from PhysicsTools.JetMCAlgos.AK4GenJetFlavourInfos_cfi import *
from PhysicsTools.JetMCAlgos.HadronAndPartonSelector_cfi import *
from PhysicsTools.JetMCAlgos.TauGenJetsDecayModeSelectorAllHadrons_cfi import *
from PhysicsTools.JetMCAlgos.TauGenJets_cfi import *

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Chihwan-An this part of the changes was already integrated for convenience at #51103.
Please remove the changes in this file from the PR.

@cmsbuild

cmsbuild commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49637/49661

@cmsbuild

cmsbuild commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Pull request #49637 was updated. @Martin-Grunewald, @cmsbuild, @mmusich can you please check and sign again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants