Remove trimmed tracking configuration for HLT Phase-2 by JanGerritSchulz · Pull Request #51816 · cms-sw/cmssw

JanGerritSchulz · 2026-09-03T09:14:40Z

PR description:

This PR is removing the trimmed tracking configuration for HLT Phase-2. This includes the procModifier phase2_hlt_vertexTrimming that activates the tracking variant, all its custom HLT module configurations, and the related runTheMatrix workflow.

This tracking option is largely outdated since newer tracking configurations like the current baseline outperform it with respect to timing and physics. Therefore from Tracking POG and HLT Upgrade sides, there is no motivation or need to keep it around any longer. Removing it simplifies maintenance and the intergration of new developments.

PR validation:

I ran a limited amount of tests locally using:

runTheMatrix.py -l ph2_hlt -i all --ibeos

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

Not a backport.

FYI @VourMa @AdrianoDee

cmsbuild · 2026-09-03T09:15:06Z

cms-bot internal usage

cmsbuild · 2026-09-03T09:16:23Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51816/50809

There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/relval_Run4.py modified in PR(s): Heterogeneous Alpaka implementation of electron pixel seed-matching #51200
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Rpc digi dev v9 #47447, Heterogeneous Alpaka implementation of electron pixel seed-matching #51200, Adding recHit masking to extended pixel tracks in heterogeneous CA #51268, Add ECAL Phase 2 development workflows with APD spikes #51636, Speed up runTheMatrix.py by up to 7x #51654, Update TenTau fragments for Phase-II workflows #51676, Add upgrade RelVal workflow for DYtoTauTautoTauLTauH #51748, [L1T] Phase2 L1Nano: drop hpsTauTable, and the dead HARVEST step from wf .781/.782 #51803, Add GEMs to FastSim #51810
- File Configuration/PyReleaseValidation/scripts/runTheMatrix.py modified in PR(s): [Do not merge] For cms-bot PR testing #48637, Heterogeneous Alpaka implementation of electron pixel seed-matching #51200
- File HLTrigger/Configuration/python/HLT_75e33/sequences/HLTPhase2PixelTracksAndVerticesSequence_cfi.py modified in PR(s): SoAMultiView #51458

cmsbuild · 2026-09-03T09:16:47Z

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

It involves the following packages:

Configuration/ProcessModifiers (operations)
Configuration/PyReleaseValidation (pdmv)
HLTrigger/Configuration (hlt)

@AdrianoDee, @DickyChant, @Martin-Grunewald, @antoniovagnerini, @cmsbuild, @davidlange6, @fabiocos, @ftenchini, @kfjack, @mandrenguyen, @miquork, @mmusich, @sroychow can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @SohamBhattacharya, @VourMa, @fabiocos, @makortel, @missirol, @mmusich, @rovere, @slomeo 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

mmusich · 2026-09-03T09:28:09Z

@JanGerritSchulz can you please tidy up also the readme file?

https://github.com/cms-sw/cmssw/blob/master/Configuration/PyReleaseValidation/README.md?plain=1#L67

cmsbuild · 2026-09-03T09:38:12Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51816/50810

There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/README.md modified in PR(s): Heterogeneous Alpaka implementation of electron pixel seed-matching #51200, Add ECAL Phase 2 development workflows with APD spikes #51636
- File Configuration/PyReleaseValidation/python/relval_Run4.py modified in PR(s): Heterogeneous Alpaka implementation of electron pixel seed-matching #51200
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Rpc digi dev v9 #47447, Heterogeneous Alpaka implementation of electron pixel seed-matching #51200, Adding recHit masking to extended pixel tracks in heterogeneous CA #51268, Add ECAL Phase 2 development workflows with APD spikes #51636, Speed up runTheMatrix.py by up to 7x #51654, Update TenTau fragments for Phase-II workflows #51676, Add upgrade RelVal workflow for DYtoTauTautoTauLTauH #51748, [L1T] Phase2 L1Nano: drop hpsTauTable, and the dead HARVEST step from wf .781/.782 #51803, Add GEMs to FastSim #51810
- File Configuration/PyReleaseValidation/scripts/runTheMatrix.py modified in PR(s): [Do not merge] For cms-bot PR testing #48637, Heterogeneous Alpaka implementation of electron pixel seed-matching #51200
- File HLTrigger/Configuration/python/HLT_75e33/sequences/HLTPhase2PixelTracksAndVerticesSequence_cfi.py modified in PR(s): SoAMultiView #51458

JanGerritSchulz · 2026-09-03T09:38:13Z

@JanGerritSchulz can you please tidy up also the readme file?

Done! Thanks, I missed that one :-)

cmsbuild · 2026-09-03T09:38:37Z

Pull request #51816 was updated. @AdrianoDee, @DickyChant, @Martin-Grunewald, @antoniovagnerini, @cmsbuild, @davidlange6, @fabiocos, @ftenchini, @kfjack, @mandrenguyen, @miquork, @mmusich, @sroychow can you please check and sign again.

mmusich · 2026-09-03T09:38:59Z

test parameters:

enable = hlt_p2_integration
workflows = ph2_hlt

mmusich · 2026-09-03T09:39:16Z

great, this comes right in time for #51817 :)

mmusich · 2026-09-03T09:39:30Z

@cmsbuild, please test

cmsbuild · 2026-09-03T11:43:10Z

+1

Size: This PR adds an extra 88KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-915f3d/55673/summary.html
COMMIT: d4d59a3
CMSSW: CMSSW_20_1_X_2026-09-02-2300/el9_amd64_gcc14
Additional Tests: HLT_P2_INTEGRATION
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/51816/55673/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 0 differences found in the comparisons
DQMHistoTests: Total files compared: 60
DQMHistoTests: Total histograms compared: 4051507
DQMHistoTests: Total failures: 0
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 4051489
DQMHistoTests: Total skipped: 18
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 59 files compared)
Checked 254 log files, 207 edm output root files, 60 DQM output files
TriggerResults: no differences found

AMD_MI300X Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 64 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 7532
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 166207
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

AMD_W7900 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 67 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 8045
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 165694
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

NVIDIA_H100 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 75 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 8674
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 165065
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

NVIDIA_L4 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 41 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 6927
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 166812
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

NVIDIA_L40S Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 85 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 8859
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 164880
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

NVIDIA_T4 Comparison Summary

Summary:

No significant changes to the logs found
Reco comparison results: 58 differences found in the comparisons
DQMHistoTests: Total files compared: 7
DQMHistoTests: Total histograms compared: 173739
DQMHistoTests: Total failures: 7126
DQMHistoTests: Total nulls: 0
DQMHistoTests: Total successes: 166613
DQMHistoTests: Total skipped: 0
DQMHistoTests: Total Missing objects: 0
DQMHistoSizes: Histogram memory added: 0.0 KiB( 6 files compared)
Checked 25 log files, 20 edm output root files, 7 DQM output files
TriggerResults: no differences found

Max Memory Comparisons exceeding threshold NVIDIA_H100

@cms-sw/core-l2 , I found 1 workflow step(s) with memory usage exceeding the error threshold:

Expand to see workflows ...

Error: Workflow 37634.7503_TTbar_14TeV+Run4D127_HLTHeterogeneousValid step2 max memory diff 221.8 exceeds +/- 30.0 MiB

Max Memory Comparisons exceeding threshold NVIDIA_L40S

@cms-sw/core-l2 , I found 1 workflow step(s) with memory usage exceeding the error threshold:

Expand to see workflows ...

Error: Workflow 37634.7503_TTbar_14TeV+Run4D127_HLTHeterogeneousValid step2 max memory diff 200.4 exceeds +/- 30.0 MiB

Max Memory Comparisons exceeding threshold NVIDIA_T4

@cms-sw/core-l2 , I found 1 workflow step(s) with memory usage exceeding the error threshold:

Expand to see workflows ...

Error: Workflow 37634.7503_TTbar_14TeV+Run4D127_HLTHeterogeneousValid step2 max memory diff -30.7 exceeds +/- 30.0 MiB

mmusich · 2026-09-03T14:38:16Z

+hlt

thanks for the clean-up

mmusich · 2026-09-07T09:40:19Z

@cms-sw/pdmv-l2 kind reminder.

antoniovagnerini · 2026-09-08T10:03:06Z

+pdmv

cmsbuild · 2026-09-08T10:03:32Z

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2)

mandrenguyen · 2026-09-09T07:48:35Z

+1

cmsbuild added this to the CMSSW_20_1_X milestone Sep 3, 2026

cmsbuild added hlt-pending operations-pending pending-signatures tests-pending orp-pending pdmv-pending code-checks-pending labels Sep 3, 2026

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 3, 2026

Remove trimmed tracking configuration for HLT Phase-2

d4d59a3

JanGerritSchulz force-pushed the jgs_rmTrimmedTracking branch from c1ef014 to d4d59a3 Compare September 3, 2026 09:36

cmsbuild added code-checks-pending and removed code-checks-approved labels Sep 3, 2026

cmsbuild added code-checks-approved and removed code-checks-pending labels Sep 3, 2026

cmsbuild added tests-started and removed tests-pending labels Sep 3, 2026

cmsbuild removed the operations-pending label Sep 3, 2026

cmsbuild added operations-approved tests-approved and removed tests-started labels Sep 3, 2026

cmsbuild added hlt-approved and removed hlt-pending labels Sep 3, 2026

mmusich mentioned this pull request Sep 3, 2026

Consider reducing the number of process modifiers active in the Phase-2 HLT menu #51817

Open

17 tasks

cmsbuild mentioned this pull request Sep 3, 2026

Add GEMs to FastSim #51810

Merged

cmsbuild mentioned this pull request Sep 7, 2026

Heterogeneous Alpaka implementation of electron pixel seed-matching #51200

Open

cmsbuild added fully-signed pdmv-approved and removed pending-signatures pdmv-pending labels Sep 8, 2026

cmsbuild added orp-approved and removed orp-pending labels Sep 9, 2026

cmsbuild merged commit f7f3640 into cms-sw:master Sep 9, 2026
30 checks passed

This was referenced Sep 9, 2026

[DEVEL] Fix dependency for extra micro-arch biglib cms-sw/cmsdist#10839

Merged

During build, correctly set LD_LIBRARY_PATH for edm check of GPU libs cms-sw/cmssw-config#121

Merged

[GCC16] Update gcc version from 16.1.1 to 16.2.1 cms-sw/cmsdist#10842

Merged

Uh oh!

Conversation

JanGerritSchulz commented Sep 3, 2026

PR description:

PR validation:

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

Uh oh!

cmsbuild commented Sep 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cmsbuild commented Sep 3, 2026

Uh oh!

cmsbuild commented Sep 3, 2026

Uh oh!

mmusich commented Sep 3, 2026

Uh oh!

cmsbuild commented Sep 3, 2026

Uh oh!

JanGerritSchulz commented Sep 3, 2026

Uh oh!

cmsbuild commented Sep 3, 2026

Uh oh!

mmusich commented Sep 3, 2026

Uh oh!

mmusich commented Sep 3, 2026

Uh oh!

mmusich commented Sep 3, 2026

Uh oh!

cmsbuild commented Sep 3, 2026

Comparison Summary

AMD_MI300X Comparison Summary

AMD_W7900 Comparison Summary

NVIDIA_H100 Comparison Summary

NVIDIA_L4 Comparison Summary

NVIDIA_L40S Comparison Summary

NVIDIA_T4 Comparison Summary

Max Memory Comparisons exceeding threshold NVIDIA_H100

Max Memory Comparisons exceeding threshold NVIDIA_L40S

Max Memory Comparisons exceeding threshold NVIDIA_T4

Uh oh!

mmusich commented Sep 3, 2026

Uh oh!

mmusich commented Sep 7, 2026

Uh oh!

antoniovagnerini commented Sep 8, 2026

Uh oh!

cmsbuild commented Sep 8, 2026

Uh oh!

mandrenguyen commented Sep 9, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

cmsbuild commented Sep 3, 2026 •

edited

Loading