Skip to content

Add Phase 2 HLT CPU vs. GPU comparison wf in default GPU tests#2710

Merged
smuzaffar merged 1 commit intocms-sw:masterfrom
VourMa:addHLTCPUvsGPUinGPUWfs
Mar 24, 2026
Merged

Add Phase 2 HLT CPU vs. GPU comparison wf in default GPU tests#2710
smuzaffar merged 1 commit intocms-sw:masterfrom
VourMa:addHLTCPUvsGPUinGPUWfs

Conversation

@VourMa
Copy link
Copy Markdown
Contributor

@VourMa VourMa commented Mar 24, 2026

Follow up on cms-sw/cmssw#50336 (comment), to make the Phase 2 HLT CPU vs. GPU workflow run by default in GPU PR tests.

@cmsbuild
Copy link
Copy Markdown
Contributor

A new Pull Request was created by @VourMa for branch master.

@akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented Mar 24, 2026

cms-bot internal usage

@mmusich
Copy link
Copy Markdown
Contributor

mmusich commented Mar 24, 2026

enable gpu

@mmusich
Copy link
Copy Markdown
Contributor

mmusich commented Mar 24, 2026

@cmsbuild, please test

@cmsbuild
Copy link
Copy Markdown
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11f2f8/52175/summary.html
COMMIT: 5f1887e
CMSSW: CMSSW_16_1_X_2026-03-23-2300/el8_amd64_gcc13
Additional Tests: GPU,AMD_MI300X,AMD_W7900,NVIDIA_H100,NVIDIA_L40S
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cms-bot/2710/52175/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially removed 5 lines from the logs
  • Reco comparison results: 9 differences found in the comparisons
  • DQMHistoTests: Total files compared: 53
  • DQMHistoTests: Total histograms compared: 4185300
  • DQMHistoTests: Total failures: 70
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 4185210
  • 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

AMD_MI300X Comparison Summary

Summary:

AMD_W7900 Comparison Summary

Summary:

NVIDIA_H100 Comparison Summary

Summary:

NVIDIA_L40S Comparison Summary

Summary:

@smuzaffar
Copy link
Copy Markdown
Contributor

+externals

this looks good. @VourMa @mmusich is it ready to go in ?

@cmsbuild
Copy link
Copy Markdown
Contributor

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. @ftenchini, @sextonkennedy, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)

@mmusich
Copy link
Copy Markdown
Contributor

mmusich commented Mar 24, 2026

is it ready to go in ?

yes, please.

@smuzaffar smuzaffar merged commit 939b747 into cms-sw:master Mar 24, 2026
21 checks passed
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.

4 participants