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 reconstruction: set fixed precision for floating point printed in unit test #33762
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33762/22713
|
A new Pull Request was created by @fabiocos (Fabio Cossutti) for master. It involves the following packages: RecoMTD/DetLayers @perrotta, @kpedro88, @cmsbuild, @srimanob, @slava77, @jpata can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test with cms-data/Geometry-TestReference#9 initial test with default architecture |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f3c380/15150/summary.html Comparison SummarySummary:
|
the tests in cms-data/Geometry-TestReference#9 show this rounding protects the unit test on other architectures, namely |
+externals |
+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) |
+1 |
PR description:
Fix rounding in printout used in unit test of
RecoMTD/DetLayers
, as the precision is creating issues with alternative architectures (PPC and arm).PR validation:
Code runs, rounding at 4 decimal digits allows check of positions at micron level. To be verified on alternative architectures.