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
update unit tests for G4e refitter package #38738
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38738/31052
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
40c164a
to
3a4e678
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38738/31056
|
Pull request #38738 was updated. @cmsbuild, @civanch, @mdhildreth can you please check and sign again. |
@cmsbuild, please test |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test testG4Refitter had ERRORS Comparison SummarySummary:
|
I noticed you took my commit and place it there. That's fine. What I am after is just fixing the technical stuff:
as promised I'll have a look this week and I can recycle this branch. |
2m limit is arbitrary. In CMS we do not have 2m empty space in central detector. |
3a4e678
to
dd89eb8
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38738/31533
|
Pull request #38738 was updated. @cmsbuild, @civanch, @mdhildreth can you please check and sign again. |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d754ac/26787/summary.html Comparison SummarySummary:
|
+1 |
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. @perrotta, @dpiparo, @qliphy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Since there are widespread plans of using the G4e refitting technology for both the muon momentum scale calibration for the W mass analysis and for the global tracker alignment procedure, it's important that we start testing it.
#38864 fixed the issues that prevented the correct testing of the package and introduced a simple analyzer for track extrapolation checks. This PR adds the consumes to the existing test analyzer and adds both of them to the unit tests of this package.
PR validation:
Relies on the add unit tests, that have been successfully exercised with
scram b runtests
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
N/A
cc:
@bendavid