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
Porting Phase2Tracker Digitizer from 62X_SLHC release. Updated also T… #13368
Porting Phase2Tracker Digitizer from 62X_SLHC release. Updated also T… #13368
Conversation
…rackerGeometry class in Geometry/TrackerGeometryBuilder moving the enum definition in the headerfile
…de EDAnalysers thread safe
… string to enum, initialisation of detector list moved to begin lumi block, used vector instead of array, pixel killing done through config parameters, testing code moved to test folder
A new Pull Request was created by @cmsbuild for CMSSW_8_1_X. It involves the following packages: Geometry/TrackerGeometryBuilder @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
please test |
The tests are being triggered in jenkins. |
+1 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
+1 |
Porting Phase2Tracker Digitizer from 62X_SLHC release. Updated also T…
Adding first version of Phase2 Tracker Digitizer, porting it from 62X SLHC release. Also updated TrackerGeometry class in Geometry/TrackerGeometryBuilder moving the enum definition in the header file.
FYI @delaere @boudoul
Automatically ported from CMSSW_8_0_X #13185 (original by @suchandradutta).