Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Const geometry #3643

Closed
wants to merge 4 commits into from
Closed

Const geometry #3643

wants to merge 4 commits into from

Conversation

VinInn
Copy link
Contributor

@VinInn VinInn commented May 2, 2014

make tracker geometry interface const
added 4 const_cast for alignment

@cmsbuild
Copy link
Contributor

cmsbuild commented May 2, 2014

A new Pull Request was created by @VinInn (Vincenzo Innocente) for CMSSW_7_1_X.

Const geometry

It involves the following packages:

Alignment/MuonAlignment
Alignment/OfflineValidation
Alignment/SurveyAnalysis
Alignment/TrackerAlignment
CalibFormats/SiStripObjects
CalibMuon/CSCCalibration
CalibTracker/SiPixelESProducers
CalibTracker/SiStripChannelGain
CalibTracker/SiStripCommon
CalibTracker/SiStripESProducers
CondFormats/SiPixelObjects
CondTools/SiPixel
CondTools/SiStrip
DQM/RPCMonitorClient
DQM/RPCMonitorDigi
DQM/SiPixelMonitorCluster
DQM/SiPixelMonitorDigi
DQM/SiPixelMonitorRecHit
DQM/SiPixelMonitorTrack
DataFormats/MuonDetId
EventFilter/SiPixelRawToDigi
Fireworks/Geometry
Geometry/CSCGeometry
Geometry/CommonDetUnit
Geometry/RPCGeometry
Geometry/TrackerGeometryBuilder
Geometry/TrackingGeometryAligner
L1Trigger/RPCTrigger
L1TriggerConfig/RPCTriggerConfig
RecoLocalMuon/RPCRecHit
RecoTracker/DeDx
SimGeneral/DataMixingModule
SimMuon/GEMDigitizer
SimMuon/RPCDigitizer
SimTracker/SiPixelDigitizer
SimTracker/SiStripDigitizer
SimTracker/TrackerMaterialAnalysis
TrackingTools/TrackAssociator
Validation/MuonRPCGeometry
Validation/RPCRecHits

@apfeiffer1, @diguida, @danduggan, @civanch, @ianna, @mdhildreth, @ktf, @cmsbuild, @alja, @anton-a, @Dr15Jones, @rcastello, @deguio, @slava77, @ggovi, @Degano, @ojeda, @rovere, @mulhearn, @thspeer, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig, @pakhotin, @frmeier, @wmtan, @GiacomoSguazzoni, @rovere, @alja, @gpetruc, @cerati, @mmusich, @venturia this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@VinInn
Copy link
Contributor Author

VinInn commented May 3, 2014

superseded by #3651

@VinInn VinInn closed this May 3, 2014
@VinInn VinInn deleted the ConStGeometry branch April 21, 2017 11:54
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.

None yet

2 participants