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
Strip hit eff update 10_2_x #22932
Strip hit eff update 10_2_x #22932
Conversation
@jlagram, CMSSW_10_2_X branch is closed for direct updates. cms-bot is going to move this PR to master branch. |
The code-checks are being triggered in jenkins. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22932/4316 Code check has found code style and quality issues which could be resolved by applying a patch in https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22932/4316/git-diff.patch You can run |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22932/4321 |
A new Pull Request was created by @jlagram for master. It involves the following packages: CalibTracker/SiStripHitEfficiency @cmsbuild, @franzoni, @arunhep, @cerminar, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+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. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
@arunhep for my understanding, in which test wf is this supposed to run, if any? Not in the basic ones as far as I can see... |
@fabiocos is not run in any relval, if that's what you mean, but I add some time ago a unit test that (among other things) runs this code as well: testProduceCalibrationTree_cfg.py |
@mmusich ok, so in practice there has been a standalone test of this PR, right? |
+1 |
@fabiocos, in practice in the PR tests themselves unfortunately nothing was run, excepted the compilation, because the package in which the test resides was not touched, but we (Tracker DPG) tested it extensively. |
@mmusich thanks for the reference documentation, I merged the code for AlCa use |
Update of the CalibTracker/SiStripHitEfficiency package used for measuring hit efficiency.
Mainly new selections added and update of tree format.