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
PLT Geometry Update #3042
PLT Geometry Update #3042
Conversation
A new Pull Request was created by @skaplanhex (Steven Kaplan) for CMSSW_7_1_X. PLT Geometry Update It involves the following packages: Geometry/ForwardCommonData @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please review it and eventually sign? Thanks. |
+1 |
@skaplanhex |
@nclopezo Did this fix the issue? Thanks! |
Pull request #3042 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please check and sign again. |
Sorry, did the rebase (hopefully it works now!) |
Pull request #3042 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please check and sign again. |
Hi, I finally heard back from Rob Loos, and it turns out there are some changes I need to make. I will do another commit soon; I apologize for all the last minute changes! |
+1 |
Sorry to keep changing. I implemented the changes I received from Rob Loos at CERN into this commit. |
Pull request #3042 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please check and sign again. |
+1 |
Geometry -- PLT Geometry Update
Update placement of sensors to account for change in PLTBCM volume. Also updates pltsens.xml to account for pixelization.