Skip to content

Conversation

@mmusich
Copy link
Contributor

@mmusich mmusich commented Oct 28, 2025

Companion PR to cms-sw/cmssw#49237

  • remove the streamers for streamDQMGPUvsCPU related to run 381594
  • add new streamers for streamDQMGPUvsCPU for run 397813
  • update the README file

Needs to be tested with cms-sw/cmssw#49237.

- remove the streamers for streamDQMGPUvsCPU related to run 381594
- add new streamers for streamDQMGPUvsCPU for run 397813
- update the README file
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mmusich for branch main.

@akritkbehera, @cmsbuild, @ctarricone, @gabrielmscampos, @iarspider, @nothingface0, @rseidita, @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
Contributor

cmsbuild commented Oct 28, 2025

cms-bot internal usage

@mmusich
Copy link
Contributor Author

mmusich commented Oct 28, 2025

test parameters:

@mmusich
Copy link
Contributor Author

mmusich commented Oct 28, 2025

@cmsbuild, please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals RelVals-INPUT
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8fd539/48872/summary.html
COMMIT: 0de073d
CMSSW: CMSSW_16_0_X_2025-10-28-1700/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-data/DQM-Integration/14/48872/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

  • 2023.0020001DAS Error
  • 2025.0000001DAS Error
  • 2022.0030001DAS Error
Expand to see more relval errors ...
  • 2024.0030001
  • 2024.0010001
  • 2024.0070001
  • 2024.0050001
  • 2025.0010001
  • 2024.0020001
  • 2024.0000001
  • 2024.0060001
  • 2024.0040001

RelVals-INPUT

  • 2023.0000001DAS Error
  • 2023.0020001DAS Error
  • 2022.0010001DAS Error
Expand to see more relval errors ...
  • 2025.0000001
  • 2022.0030001
  • 2024.0030001
  • 2024.0010001
  • 2025.0010001
  • 2024.0070001
  • 2024.0050001
  • 2025.0020001
  • 2023.0010001
  • 2022.0000001
  • 2022.0020001
  • 2024.0020001
  • 2024.0000001
  • 2024.0060001
  • 2024.0040001

@mmusich
Copy link
Contributor Author

mmusich commented Oct 29, 2025

please test

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8fd539/48876/summary.html
COMMIT: 0de073d
CMSSW: CMSSW_16_0_X_2025-10-28-2300/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-data/DQM-Integration/14/48876/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8fd539/48876/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8fd539/48876/git-merge-result

Comparison Summary

Summary:

  • You potentially removed 54 lines from the logs
  • Reco comparison results: 8 differences found in the comparisons
  • DQMHistoTests: Total files compared: 51
  • DQMHistoTests: Total histograms compared: 3939953
  • DQMHistoTests: Total failures: 15895
  • DQMHistoTests: Total nulls: 22
  • DQMHistoTests: Total successes: 3924016
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 50 files compared)
  • Checked 218 log files, 188 edm output root files, 51 DQM output files
  • TriggerResults: found differences in 4 / 49 workflows

@cmsbuild
Copy link
Contributor

REMINDER @ftenchini, @mandrenguyen, @sextonkennedy: This PR was tested with cms-sw/cmssw#49237, please check if they should be merged together

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2025

REMINDER @ftenchini, @sextonkennedy, @mandrenguyen: This PR was tested with cms-sw/cmssw#49237, please check if they should be merged together

@ftenchini
Copy link

+1

@ftenchini
Copy link

@nothingface0 could we please have an approval here as well? Thanks.

@nothingface0
Copy link

nothingface0 commented Nov 3, 2025

+dqm
Sorry, missed it!

@mmusich
Copy link
Contributor Author

mmusich commented Nov 3, 2025

@smuzaffar @iarspider can you sign for externals? We'll need also a 15.1.x backport for the next darataking release.

@smuzaffar
Copy link
Contributor

+externals

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2025

This pull request is fully signed and it will be integrated in one of the next main IBs (tests are also fine). This pull request will be automatically merged.

@cmsbuild cmsbuild merged commit f760697 into cms-data:main Nov 3, 2025
10 checks passed
smuzaffar added a commit to cms-sw/cmsdist that referenced this pull request Nov 3, 2025
@mmusich mmusich deleted the pixel_dqm_client_changes_for_CMSHLT-3147 branch November 4, 2025 01:18
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.

5 participants