Skip to content

Phase2-hgx367 Try to create a scenario with T37, C27, N17, I22, F9 - #51080

Merged
cmsbuild merged 4 commits into
cms-sw:masterfrom
bsunanda:Phase2-hgx367
Jun 11, 2026
Merged

Phase2-hgx367 Try to create a scenario with T37, C27, N17, I22, F9#51080
cmsbuild merged 4 commits into
cms-sw:masterfrom
bsunanda:Phase2-hgx367

Conversation

@bsunanda

Copy link
Copy Markdown
Contributor

PR description:

Try to create a scenario with T37, C27, N17, I22, F9

PR validation:

Working with the new GEM geometry builder

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:

To be seen if it is to be back ported> Debug is incomplete

@cmsbuild

cmsbuild commented May 28, 2026

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-51080/49536

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@cmsbuild

Copy link
Copy Markdown
Contributor

@cmsbuild

Copy link
Copy Markdown
Contributor

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

It involves the following packages:

  • Configuration/Geometry (geometry)
  • Configuration/PyReleaseValidation (pdmv)
  • Configuration/StandardSequences (operations)
  • Geometry/CMSCommonData (geometry)
  • Geometry/GEMGeometryBuilder (geometry)

@AdrianoDee, @DickyChant, @Dr15Jones, @antoniovagnerini, @bsunanda, @civanch, @cmsbuild, @davidlange6, @fabiocos, @ftenchini, @kfjack, @kpedro88, @makortel, @mandrenguyen, @mdhildreth, @miquork, @sroychow can you please review it and eventually sign? Thanks.
@GiacomoSguazzoni, @Martin-Grunewald, @VinInn, @VourMa, @dgulhan, @elusian, @fabiocos, @felicepantaleo, @giovanni-mocellin, @jshlee, @makortel, @martinamalberti, @mmasciov, @mmusich, @mtosi, @rovere, @sameasy, @slomeo, @vargasa, @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

* M14: Phase2 muon system for TDR w/ GE2/1, GE0, RE3/1, RE4/1 (incl. granularity in ME0, staggered GE2/1), 96 iRPC strips; no overlaps, MB4Shields with right value for YE3 size and Shield structure modified in muonYoke, changed number of strips and corrected eta partition size for GE21, a realistic support structure for GE0 and adjustment for boundaries, right front-back relation between alternate phi segments
* M15: same as M14 but removing overlaps in yoke, MB3, GE0 + adding DT shield
* M16: same as M15 with reverting RPC endcap disk4 rotation
* M17: same as M15 with the correct shape of GE0

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.

Please add the definition of D126 at the bottom of this README file

@bsunanda

bsunanda commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

@cmsbuild Please test

@cmsbuild cmsbuild modified the milestones: CMSSW_17_0_X, CMSSW_20_0_X Jun 5, 2026
@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))

@cmsbuild

cmsbuild commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

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

@cmsbuild cmsbuild modified the milestones: CMSSW_20_0_X, CMSSW_20_1_X Jun 9, 2026
@bsunanda

bsunanda commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Can this be merged urgently?

@AdrianoDee

Copy link
Copy Markdown
Contributor

+pdmv

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

@ftenchini

Copy link
Copy Markdown
Contributor

+1

@cmsbuild
cmsbuild merged commit 8501240 into cms-sw:master Jun 11, 2026
10 checks passed
NJManganelli pushed a commit to NJManganelli/cmssw that referenced this pull request Jul 25, 2026
Phase2-hgx367 Try to create a scenario with T37, C27, N17, I22, F9
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