Skip to content

[GEM][Backport] Introducing GE21 rechit container for Run 3#50557

Open
yeckang wants to merge 1 commit intocms-sw:CMSSW_16_0_Xfrom
yeckang:ge21Container_16_0_X
Open

[GEM][Backport] Introducing GE21 rechit container for Run 3#50557
yeckang wants to merge 1 commit intocms-sw:CMSSW_16_0_Xfrom
yeckang:ge21Container_16_0_X

Conversation

@yeckang
Copy link
Copy Markdown
Contributor

@yeckang yeckang commented Mar 26, 2026

PR description:

  • This PR enables saving the GEMRecHits from GE21 chambers, which are installed for testing purposes.
    Currently, we are not saving them because of the possibility of interruption in the muon reconstruction performance.
    However, several groups studying GE21 performance continue to ask if we can save them.
    For this reason, GEM DPG decides to save them in a separate rechit container, which does not propagate to the muon reco system.

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:

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented Mar 26, 2026

A new Pull Request was created by @yeckang for CMSSW_16_0_X.

It involves the following packages:

  • DQM/Integration (dqm)
  • HLTrigger/Configuration (hlt)
  • RecoLocalMuon/GEMRecHit (reconstruction)

@Martin-Grunewald, @Moanwar, @cmsbuild, @ctarricone, @gabrielmscampos, @jfernan2, @mandrenguyen, @mmusich, @nothingface0, @rseidita, @srimanob can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @batinkov, @bellan, @francescobrivio, @jhgoh, @jshlee, @missirol, @mmusich, @threus, @watson-ij 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

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented Mar 26, 2026

cms-bot internal usage

@gabrielmscampos
Copy link
Copy Markdown
Member

test parameters:

  • addpkg = DQM/Integration

@gabrielmscampos
Copy link
Copy Markdown
Member

please test

@gabrielmscampos
Copy link
Copy Markdown
Member

backport of #49476 #50556

MinOneOverPtError=cms.double(1.0),
TTRHBuilder=cms.string("hltESPTTRHBWithTrackAngle"),
magneticField=cms.string("ParabolicMf"),
beamSpot=cms.InputTag("hltOnlineBeamSpot")
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.

What is this removal about? It does not seem part of the master PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, maybe I made a mistake when I copied the file from the old workspace. I will revert and push it again

@cmsbuild
Copy link
Copy Markdown
Contributor

+1

Size: This PR adds an extra 24KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9a8247/52280/summary.html
COMMIT: b81347e
CMSSW: CMSSW_16_0_X_2026-03-25-2300/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/50557/52280/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially removed 3 lines from the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 2 differences found in the comparisons
  • DQMHistoTests: Total files compared: 53
  • DQMHistoTests: Total histograms compared: 4166625
  • DQMHistoTests: Total failures: 75
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 4166530
  • 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
Copy link
Copy Markdown
Contributor

mmusich commented Mar 26, 2026

-hlt

@cmsbuild
Copy link
Copy Markdown
Contributor

Pull request #50557 was updated. @Martin-Grunewald, @Moanwar, @cmsbuild, @ctarricone, @gabrielmscampos, @jfernan2, @mandrenguyen, @mmusich, @nothingface0, @rseidita, @srimanob can you please check and sign again.

@yeckang
Copy link
Copy Markdown
Contributor Author

yeckang commented Mar 27, 2026

please test

@cmsbuild
Copy link
Copy Markdown
Contributor

+1

Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9a8247/52304/summary.html
COMMIT: 1277b98
CMSSW: CMSSW_16_0_X_2026-03-26-2300/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/50557/52304/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially added 3 lines to the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 1 differences found in the comparisons
  • DQMHistoTests: Total files compared: 53
  • DQMHistoTests: Total histograms compared: 4166625
  • DQMHistoTests: Total failures: 3
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 4166602
  • 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
Copy link
Copy Markdown
Contributor

mmusich commented Mar 27, 2026

backport of #49476

@mmusich
Copy link
Copy Markdown
Contributor

mmusich commented Mar 27, 2026

backport of #50556

@mmusich
Copy link
Copy Markdown
Contributor

mmusich commented Mar 27, 2026

+hlt

@gabrielmscampos
Copy link
Copy Markdown
Member

+dqm

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