Skip to content

Conversation

@emiglior
Copy link
Contributor

@emiglior emiglior commented Aug 2, 2023

PR description:

This PR is the first step towards the description of the so-called big pixels for the phase-2 InnerTracker as proposed in

https://indico.cern.ch/event/1242584/contributions/5517916/attachments/2691752/4671198/em20230728.pdf

The plan is to describe the phase-2 big pixels in the file pixelStructureTopology.xml or by using the TrackerAdditionalParametersPerDet rcd for geometries loaded from DB.

The goal of this PR is to put back the changes of the infrastructure needed for using the TrackerAdditionalParametersPerDet rcd, introduced for the bricked pixels and then removed with PR#40443 (but without reintroducing the functions related to bricked pixels).

PR validation:

  • compilation ok after git cms-checkdeps -a
  • scram b code-checks, code-format
  • checked on ph-2 w/f 25610

This PR is not a backport and no backport needed

@tvami @mmusich @adewit @gbardell

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 2, 2023

A new Pull Request was created by @emiglior (Ernesto Migliore) for CMSSW_13_2_X.

It involves the following packages:

  • Alignment/CommonAlignmentMonitor (alca)
  • Alignment/CommonAlignmentProducer (alca)
  • Alignment/LaserAlignment (alca)
  • Alignment/MillePedeAlignmentAlgorithm (alca)
  • Alignment/OfflineValidation (alca)
  • Alignment/SurveyAnalysis (alca)
  • Alignment/TrackerAlignment (alca)
  • CondTools/Geometry (db)
  • Configuration/Geometry (geometry, upgrade)
  • Geometry/TrackerGeometryBuilder (geometry)

@perrotta, @consuegs, @Dr15Jones, @bsunanda, @makortel, @civanch, @mdhildreth, @cmsbuild, @AdrianoDee, @srimanob, @saumyaphor4252, @tvami, @francescobrivio can you please review it and eventually sign? Thanks.
@pakhotin, @VourMa, @vargasa, @adewit, @JanFSchulte, @tocheng, @VinInn, @Martin-Grunewald, @missirol, @bsunanda, @tlampen, @mmusich, @mtosi, @fabiocos, @slomeo, @venturia this is something you requested to watch as well.
@perrotta, @dpiparo, @rappoccio you are the release manager for this.

cms-bot commands are listed here

@tvami
Copy link
Contributor

tvami commented Aug 2, 2023

hi @emiglior the master release in CMSSW is now 13_3_X, can you please make the PR there too? (CMSSW policy is to have PRs in master first, and then the backports)

@emiglior
Copy link
Contributor Author

emiglior commented Aug 2, 2023

Thanks @tvami for pointing it out!
I have created PR #42454 to supersede this one

@emiglior emiglior closed this Aug 2, 2023
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.

3 participants