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
MTD geometry and digitization: move check for pixel active area into RectangularMTDTopology, make it usable for BTL as well #33453
Conversation
… usable for BTL as well
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33453/22150
|
A new Pull Request was created by @fabiocos (Fabio Cossutti) for master. It involves the following packages: Geometry/MTDCommonData @civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @srimanob, @kpedro88 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test GeometryMTDGeometryBuilderTestDriver had ERRORS Comparison SummarySummary:
|
the failure in the |
please test with cms-data/Geometry-TestReference#7 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-acf605/14304/summary.html Comparison SummarySummary:
|
@parbol FYI |
+1 |
+1 |
+Upgrade |
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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
@silviodonato @qliphy @mrodozov please recall to merge this together cms-data/Geometry-TestReference#7 , thanks |
asap as this is merged i'll get the data external |
+1 |
PR description:
#33340 has cleaned the
RectangularMTDTopology
and introduced functionalities to take into account non-active areas around pixels. This has been used in the ETL digitization, but it is a general functionality potentially valid fro BTL as well. While in BTL the crystals are correctly positioned at the center of pixels, and there is so far no use of inter-crystal gaps, this PR turns the check for ETL into a general method usable also for BTL if needed.In order to do this, the
mtdParameters.xml
values are updated for BTL with values extracted from the ideal geometry, rounding at 1 micron level, and for y moving a residual 0.5 inter-crystal micron to the border gap.PR validation:
Test wf 34634.0 runs.