-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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: fix RecoMTD/DetLayers unit test #45310
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45310/40724
|
A new Pull Request was created by @fabiocos for master. It involves the following packages:
@cmsbuild, @jfernan2, @mandrenguyen, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test with cms-data/Geometry-TestReference#15 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3a4ff4/40083/summary.html
Comparison SummarySummary:
|
please test with cms-data/Geometry-TestReference#15 for el8_aarch64_gcc12 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3a4ff4/40103/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: |
type mtd |
+1 |
@cms-sw/upgrade-l2 @cms-sw/orp-l2 this PR (and the corresponding cms-data/Geometry-TestReference#15 ) will prevent the unit test crash in IB observed for aarch64 architecture, reported by @aandvalenzuela |
+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. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
cms-data/Geometry-TestReference#15 merged. |
+1 |
PR description:
As a consequence of the move of all MTD geometry unit tests to scenario D110 in #45211 , the unit test for the tracking geometry navigation in
RecoMTD/DetLayers
starts to fail on aarch64 architecture #45211 (comment) , due to some numerical differences in the output of operations displayed in the reference file.The underlying geometry calculations are made using
float
for performance reasons, and on operations involving space measurements of a size O(100 cm) does not always guarantee to display identical results at the micron level on both amd64 and aarch64. For this reason:PR validation:
The unit test comparison succeed on both amd64 and aarch64 (lxplus-arm.cern.ch).